-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
withings_sync.garmin.APIException: SSO error 401 #31
Comments
Are you on the latest version? Have you see the #24 ? Best regards, Jarek |
Starting to have problems too since today. I'm on the latest commit. |
I noticed the same, but sporadically. Sometimes Garmin/Cloudflare decides to throw the 'maintenance' error, but eventually my measurements are synced. I am not sure if it is an actual maintenance on the Garmin platform or if it's the security implementation that is blocking the scripts. |
Could we fix this with some retry logic? |
See also #26 |
@stv0g At least for me, a retry logic is probably irrelevant, since it's 2 days that I keep having the same problem (also, the script is run 1 time per day, except for some manual run yesterday and today, so I don't think I got banned). But I guess I'm hijacking this issue since it might be a different thing |
We could try a retry logic, but I am a bit hesitant to test with it. IF this is the CloudFlare security implementation I won't be surprised if you actually get IP banned when retrying too aggressively. I am also not sure about the Garmin API throttling. What works for me is to schedule the sync every 2hrs. For now that seems to be the sweet spot. edit: the scheduled script at 10:00 this morning bugged out with an SS0 403 error code. A manual run 40 minutes later ran successfully. |
Thanks for the feedback @stynoo and @m3l7.
I will add a note to the README.
I would be nice if we could solve #3 . But I dont think Garmin really supports this currently. Alternatively, we should probably also try to cache our Garmin tokens to avoid a new SSO login every-time the script runs. |
This still happens to me, tried a couple of different IP addresses to rule out being caused by a ban but no joy. I have it as a cron job to run once per day. Sometimes it works but it seems to be random and very low probability. On top of this #23 exacerbates the problem, as a rare successful update won't back-fill any of the previously failed ones. I am on v3.3.1 |
Also having this issue on the very latest code. Haven't been able to get a single good sync since installing this. |
I have the same issue. Running a fresh installation. Got 401 on the first sync. |
Leaving this comment if someone comes across this error. In my case, my password had a '&' and hence it was escaped. so you will have to include your password in double quotes. "mypassword&withampersand" |
The text was updated successfully, but these errors were encountered: