Skip to content

Commit f950306

Browse files
matinclaude
andcommitted
Remove SSO_PAGE_HEADERS from login POST and MFA verify
PR #215 re-introduced browser headers on the login POST and MFA verify endpoints. These are mobile API endpoints that expect the default GCM-iOS user agent — browser headers cause 401/429 from Garmin. Same regression as 0.7.7, fixed in 0.7.8, re-introduced in 0.7.10. Browser headers should ONLY be on page requests (sign-in, embed). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 6c1b537 commit f950306

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

src/garth/sso.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,6 @@ def login(
118118
"sso",
119119
"/mobile/api/login",
120120
params=login_params,
121-
headers=SSO_PAGE_HEADERS,
122121
json={
123122
"username": email,
124123
"password": password,
@@ -218,7 +217,6 @@ def handle_mfa(
218217
"sso",
219218
"/mobile/api/mfa/verifyCode",
220219
params=login_params,
221-
headers=SSO_PAGE_HEADERS,
222220
json={
223221
"mfaMethod": mfa_method,
224222
"mfaVerificationCode": mfa_code,

src/garth/version.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
__version__ = "0.7.11"
1+
__version__ = "0.7.12"

0 commit comments

Comments
 (0)