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

Avoid some errors in the log #278

Merged
merged 1 commit into from
Dec 13, 2023
Merged

Avoid some errors in the log #278

merged 1 commit into from
Dec 13, 2023

Conversation

jnxxx
Copy link

@jnxxx jnxxx commented Dec 13, 2023

It seems there is a minor bug when refreshing the access_token from nestauthproxyservice.
As input to the authenticate method it gives the entire auth object rather than the access_token within it. Thus, the call to nestauthproxyservice will fail.

It seems to overcome the issue by trying again later, apparently with correct input.
This PR just avoids some errors in the log. Most likely the ones mentioned in #267.

@jnxxx jnxxx requested a review from iMicknl as a code owner December 13, 2023 13:33
@iMicknl
Copy link
Owner

iMicknl commented Dec 13, 2023

Thanks @jnxxx!

@iMicknl iMicknl merged commit 9f94f36 into iMicknl:beta Dec 13, 2023
3 checks passed
iMicknl pushed a commit that referenced this pull request Jan 27, 2024
Co-authored-by: jnxxx <>
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.

2 participants