Should check status code from requests.get() first in download_mod() about fac HOT 3 CLOSED

There's no way it wouldn't fail later in the process anyway, since the zip file is opened and info.json is read before the zip gets saved.

Anyway, I added some proper error messages for both the "already logged in with no game ownership" and also added the require_ownership=true parameter which the game uses on login. This is what it looks like now:

$ fac install yarm
Installing: YARM 0.7.107...
Downloading: https://mods.factorio.com/api/downloads/data/mods/187/YARM_0.7.107.zip...
Authentification error when downloading mod. Please login again.
You need a Factorio account to download mods.
Please provide your username and password to authenticate yourself.
Your username and token (NOT your password) will be stored so that you only have to enter it once
This uses the exact same method used by Factorio itself

Username [mickael9-test]:  
Password (not shown):
Ownership error: Your factorio account doesn't own the game.
Please buy the game or link your Steam account if you have bought the game from Steam.

from fac.

You can use -v for requests debugging :)

I'm not having this issue, I suspect the 403 error is because your token or username is invalid (from player-data.json)
Are you able to download mods from the game?

Anyway, I'll add some error handling so that invalid error codes are raised as HTTPError

from fac.

See edit to the OP :)

Anyway, I'll add some error handling so that invalid error codes are raised as HTTPError

Yeah, that's what I was asking for. Otherwise there's the possibility a non-200 response has the same content length as the actual mod file, and fac install won't notice.

from fac.