-
Notifications
You must be signed in to change notification settings - Fork 81
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
Constant authentication requests since last update #1046
Comments
Same here, but I have nothing in the logs. My config: |
Arlo changed the backend around. Can you upload some logs?
…On Thu, Feb 13, 2025, 04:48 Gary Brown ***@***.***> wrote:
Since the update I am getting constant emails notify me of one time code
and new login almost every minute! Has something changed in the last
updates? Is there a way to stop this?
—
Reply to this email directly, view it on GitHub
<#1046>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALHJU6RYXCZKXZXTTNOA5DT2PRS7VAVCNFSM6AAAAABXBXEB2CVHI2DSMVQWIX3LMV43ASLTON2WKOZSHA2TANJRHE4TIMQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
home-assistant_2025-02-13T09-59-20.066Z.log This is my latest log file - I have rolled back to 0.8.1.11 at the end of the logs to try and remediate just in case this looks weird! |
I am not at home was only wondering why there are so many mails. |
The rollback seems to have stopped the mails for now so could be the user agent change causing arlo to think theres a new device all the time? |
The issue seems to have started about 06:52 this morning. |
It started for me at 4 in the morning. I have verbose debugging on and I'm seeing this error:
It looks like they've added a timeout to the event stream. A work around might be to add I'll look into this some more. |
Will try, thanks. |
They've changed the general timeouts to 30 minutes and this includes the access token. It used to be 2 weeks. This is the I need to modify the login code to "trust" the Arlo session and stop the emails happening - this should be possible. For now I think you are going to get those messages every half an hour. I'll take a look at it today. |
I think the value 25 had no effect. Thanks for your work. |
I realised you're still going to get the emails but it should have stopped the errors. |
Just realized they change the access token expiration to 2 hours. This is what I got when inspecting the login from an incognito window. So Im sure this is not related to how we mock the payload
First of all this will break the intention of the check session we do now: Secondly, not sure if this is related to many issues we've been experiencing recently. |
@yongfg that is exactly the problem The way this used to work wasOn the first login:
For logins with in the next 12 days:
For logins within the 12 days we didn't need to ask for a 2fa code. What changedBut with the new back end changes arlo has reduced the token timeout to 2 hours (it was actually 30 minutes earlier today). So once that token expires we need to get use 2fa to get another token. What I'm looking atI'm testing a pyaarlo branch that makes the component mock the web browser even more closely. With the new changes: On the first login
On subsequent logins:
The branch is working pretty well but I'll run some more test tonight. Once this workaround is in place I'm going to revisit and refactor this code, it started off simple and has morphed into quite the lump of spaghetti code over time. |
Haha totally, I was also messing the code in the similar direction but got no luck. I tried to inspect if they have some token exchange logic or if this field Looking forward to the fix and I'm willing to help test and clean up if needed! Thank you |
Im seeing a new update for the pyaarlo repo. @twrecked Thank you for the push! I'm wondering if there's an efficient way to test this. I kinda don't want to always wait for 2 hours before figuring out if the changes are working. Also for the new approach, does it matter if we have to be consistent on the user agent? |
@yongfg you can set The disconnect mechanism is different but the overall effect is the same. |
This should be fixed in v0.8.1.15 if people want to give it a try. I've had to modify the authentication code and I've tested as thoroughly as I can on my setups. But experience has shown Arlo to be an odd beast so if you run into issues please post some debug logs here. You can return to v0.8.1.41.1 and it will use the old authentication mechanism. |
@twrecked From my understanding of the code, it looks like we always try to do auth with username and password. And for the happy path, we don't have to do 2FA. My question is, does it mean that at least the username and password should always be available? Like we can't silently refresh the access token without the username and password? |
Good questions. My goal here was to get it working but it's strained the current code to breaking point. So I don't know the answer just yet but I suspect as long as the browser token is valid we might be able to skip to login. But I'm just guessing. Once I've tidied the state machine I should have a better understanding and I'll give it a try. |
I'm still getting regular app approval prompts in less than an hour apart with .15... |
@Phoenix-DH did you find 15? You might need to get HACS to update its information @hceuterpe Can you attach some logs. |
Yes, just did the update and removed the lines in the config. Thanks. |
@Phoenix-DH excellent, let me know how it goes |
Again up and running 👌🏻 Restart 11:16am. |
@hceuterpe At the computer now...
edit: you are using the push mechanism? I just realised that I'm not setting the trusted browser on that. The code has 2 paths that do essentially the same thing. I'll fix it. For now you if you have set |
Until now …. No mail … still working. |
@hceuterpe can you try the 16 release. It might/should fix it. |
Since the update I am getting constant emails notify me of one time code and new login almost every minute! Has something changed in the last updates? Is there a way to stop this?
The text was updated successfully, but these errors were encountered: