-
Notifications
You must be signed in to change notification settings - Fork 31
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
Authentication issues with direct uploading TCX files #22
Comments
These users are on the latest garmin-uploader-1.0.8 version. We seem to have a 50/50 split of people for whom the 1.0.8 version fixed the auth issue (it works for me) and people who are still seeing auth issues. |
I'm receiving the same error as @akramer115. 2021-01-11 08:51:46,372 [INFO] Try to login on GarminConnect... I have no issues signing into the Garmin website. |
upgrading garmin-uploader-1.0.8 fixed the issue for me. |
Hello guys, I read the issue on the peloton-to-garmin project from @philosowaffle , looks like it works... sometimes. I can't reproduce the issue with several accounts of my own. This may be due to the fact that i hardcode some french localization flags (and Garmin would detect it makes little sense by geo-locating your ip). The only way for me to debug that issue is if some of you who are affected provide me with their Garmin credentials (after changing the password). I know it's tricky to ask that, but I don't see another easy way |
Hi @La0. Thanks so much for the assistance. I can email you my sign on information to test with. |
@amy26point2 Ok, you can send me an email on bastien@nextcairn.com Here is my GPG public key if you want to send your credentials through an encrypted email. |
@amy26point2 I got your credentials, and they seem to work from my computer 😢 I'll upload a patch to add some diagnostic tooling, so you can try on your own computer. I'll update that issue once i've got something |
That is strange. I half expected it to work after I changed my credentials, but I was still getting an error when I tested it. I'll run whatever you need from me. Thanks again! |
Have any of you tried the windows binary .zip? It worked for me after the last updates. Might help with any incompatibilities associated with python versions. |
This worked for a few weeks then started failing. This is awesome by the way! EDIT: Auth started failing then I upgraded my uploader and auth is still failing, few more details below: Old: pip install garmin-uploader --upgrade New: gupload -u xyz@yahoo.com -p xxxxx -t "running" Output/file.tcx file.tcx was created by peloton->Garmin uploader - I renamed it for simplicityRunning on Raspberry pi 3 |
Unfortunately 1.0.8 is still broken as well.
Jason
From: Bastien Abadie <notifications@github.com>
Sent: Tuesday, January 12, 2021 1:06 PM
To: La0/garmin-uploader <garmin-uploader@noreply.github.com>
Cc: WhatevsJJ <jasonj@squarej.com>; Mention <mention@noreply.github.com>
Subject: Re: [La0/garmin-uploader] Authentication issues with direct uploading TCX files (#22)
@WhatevsJJ <https://github.com/WhatevsJJ> Version 1.0.7 is expected to fail. Please upgrade to the latest version of garmin uploader
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#22 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/ASOE77BFHX23GL3ZNT76VJDSZSMTBANCNFSM4V4RWI3Q> .
|
Okay guys, I definetely cannot reproduce the issue on my side, i tried with different servers at different locations: no luck. Garmin may be sending you different parameters when you login. So we need to use the big guns to solve that issue 🔫 Now bear with me:
If you want to help, you'll need to :
Thanks for helping, I know it's quite the struggle... |
Interesting, I just now created a brand new connect.garmin account to test further and it worked fine on a different computer (VM). So then I tried my real account to upload data and I'll be damn, its working now too as is my wife's account. BOTH were failing earlier today. No idea what changed. Thanks for this great module. |
Just sent you my HAR file via WeTransfer - let me know if you need anything
else. It might be worth noting that I am using a Mac computer, running Big
Sur, version 11.1.
…On Tue, Jan 12, 2021 at 9:04 PM WhatevsJJ ***@***.***> wrote:
Interesting, I just now created a brand new connect.garmin account to test
further and it worked fine on a different computer (VM). So then I tried my
real account to upload data and I'll be damn, its working now too as is my
wife's account. BOTH were failing earlier today. No idea what changed.
Thanks for this great module.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#22 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASNC6BZZVCDL23ENIZELSB3SZT5SXANCNFSM4V4RWI3Q>
.
|
Hello guys, I received your HAR files, thanks for your help. I did not see some big difference with the current auth, but i made a patch you can try:
This uses |
That worked! Thank you so much for your help!!
…On Thu, Jan 14, 2021 at 2:26 PM Bastien Abadie ***@***.***> wrote:
Hello guys,
I received your HAR files, thanks for your help. I did not see some big
difference with the current auth, but i made a patch you can try:
pip install https://github.com/La0/garmin-uploader/archive/us-auth.zip
This uses en-us locales, and a few parameters changes, let's see if that
works out for you...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#22 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASNC6B5R33ENKEPE7KMTOW3SZ5AONANCNFSM4V4RWI3Q>
.
|
When I performed the install is said all the requirements were already satisfied, so it doesn't look like it changed anything. The error still occurs. Collecting https://github.com/La0/garmin-uploader/archive/us-auth.zip |
Add me to the list of people it just suddenly started working for again. Thanks so much for the effort! |
Despite this still being a bit of a mystery thanks for going above and beyond to help out @La0. Is there a link we can buy you a coffee at? |
I've just started hitting on this problem after successfully using garmin-uploader for the last month or so. I've tried the us-auth branch to no avail. I've started looking into it and added some extra debugging information and got this unexpected result for the headers of the response
|
I think Garmin pushed a change this afternoon, I also started getting auth failures out of the blue. |
Yes, had intermittent issues for a few weeks and then from yesterday it failed every time. us patch applied and tested, still not working. Garmin must have 'done something'... From looking at the data it is the session.get(URL_PROFILE) that is failing and showing cloudflare details i.e. garmin are using cloudflare so it is not getting to the API endpoint. Had a quick google and looks like there is this: |
Actually it is the profile fetch after logging in that fails. If you comment out as below it works.
|
Thanks guys for the report and finding a solution ! I won't be able to make a patch today, so if one of you want to contribute a Pull Request, i'd gladly review it. I think we can use |
I tried that URL, and it seems to work. You can test that patch by doing:
If that patch works for you, i'll merge it and make a new release of garmin-uploader |
Ok, that seems to have fixed it. Thanks for the update :) |
Thanks. That fixed it for me. I'm using your library in my "iFit to Garmin" web app. https://github.com/flackdl/ifit-garmin-sync |
It works here too! Thanks a lot! 👍 |
Hey all -
Hmm I installed the test patch but am still receiving an authentication
error:
2021-02-25 07:35:30,858 - INFO - peloton-to-garmin: Uploading activities to
… Garmin...
2021-02-25 07:35:32,795 [CRITICAL] Login Failure: Login check failed.
Traceback (most recent call last):
File "/Users/AK/Desktop/peloton-to-garmin/peloton-to-garmin.py", line
95, in <module>
PelotonToGarmin.run(config)
File "/Users/AK/Desktop/peloton-to-garmin/peloton-to-garmin.py", line
82, in run
raise e
File "/Users/AK/Desktop/peloton-to-garmin/peloton-to-garmin.py", line
79, in run
garminUploader.uploadToGarmin(garminUploadHistoryTable)
File "/Users/AK/Desktop/peloton-to-garmin/lib/garminClient.py", line
29, in uploadToGarmin
assert self.user.authenticate(), "Failed to authenticate garmin user."
AssertionError: Failed to authenticate garmin user.
On Thu, Feb 25, 2021 at 6:53 AM Aurélien Grosdidier < ***@***.***> wrote:
It works here too! Thanks a lot! 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#22 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASNC6B44ZM45POTOXWFMDSTTAY23ZANCNFSM4V4RWI3Q>
.
|
Thanks for your help, I just released garmin-uploader 1.0.9 with this patch. You can now upgrade your installations:
|
Hi there-
I'm using the Peloton-to-garmin project to convert Peloton workouts to TCX files with direct uploading to Garmin. We've been experiencing issues with authentication to Garmin - wondering if others are too. I'm receiving this error (and I've triple checked credentials):
The text was updated successfully, but these errors were encountered: