Skip to content
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

fixed __get_login_session 504 gateway timeout #39

Merged
merged 1 commit into from
Jun 29, 2023
Merged

fixed __get_login_session 504 gateway timeout #39

merged 1 commit into from
Jun 29, 2023

Conversation

sedy89
Copy link
Contributor

@sedy89 sedy89 commented Jun 28, 2023

I had the same issue that was reported by other users Cant sign-in #38
The configuration window of the integration was showing username/password issues.

After some tesing I was able to pass the login sequence by deleting the predefined common headers in api.py.

Seems like there is no other change required.
Would be nice if someone could cross check...
BR

@sedy89 sedy89 requested a review from yo-han as a code owner June 28, 2023 16:55
@Mister86NL
Copy link

Gevalideerd, account werd direct geaccepteerd. @yo-han @sedy89

@yo-han
Copy link
Owner

yo-han commented Jun 29, 2023

Yes, works on my machine.

@yo-han yo-han merged commit 9ba958e into yo-han:develop Jun 29, 2023
@yo-han
Copy link
Owner

yo-han commented Jun 29, 2023

It's good to keep an eye on this issue because I'm not completely sure why these headers are an issue suddenly. It can be something like Medtronic blocking these requests or a switch in response types use by the Medtronic clients.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants