denniedegroot / com.ring

Ring for Homey
https://www.athom.com
Other
15 stars 15 forks source link

Error picking up and sending ring screenshot #59

Open erwinvang opened 4 years ago

erwinvang commented 4 years ago

643866E8-AB41-4F16-8488-2CA503BD3FB8

I want ring to grab screenshot when doorbell is pressed. And than push it to my phone.

It seems to work once or twice but after that i get attached error. Have you seen this error? Do you know where I should look? I confirmed the 2 step verificarion already.

Thanks in advance

kaohlive commented 4 years ago

No, very strange indeed if it works a couple of times. What it tells us is that the access token we get from the MFA process is not valid for this resource. The resource in this case being the API we call. So it almost seems that the code is sending a refresh token or access token that is not generated by the ring api. But can only be the case if you pasted one directly into the settings using the dev portal or something. Or that it indeed was revoked... that could be done by someone or Ring themselves. Then you have to reauthenticate. Maybe you got a mail that they noticed a new login and you pressed the that wasnt me kind of link?

Kingster2 commented 4 years ago

I didn’t get a email from ring. For now I had to remove the doorbell in Homey and reinstall, because this is the only way to reauthenticate. All attached flows had to be updated. Now it works for a few days without problems.

Before we could reauthenticate in the app settings without the need of updating flows. Is there a way to easily reauthenticate. Probably most users need to reauthenticate once in a while.

kaohlive commented 4 years ago

@Kingster2 you actually still can. You should not have to remove the device. You can just use the revoke button in the settings and then go through the add device, authenticate wizard. If you did not removed the device it should just say that there are no new device but it should have a fresh token.

I know the current wizard is not all that intuitive, but it was my only way to solve the more complex steps needed due to the MFA, that easy done in that wizard and a lot of work in the app settings location. But I do have more understanding on how that works now. I need to stabilize the current version with the new device feature support, the MFA and new homey image objects to fix the snapshot grabbing. And since its not really my app, i just contribute at this time, I have some other apps to support also. So making the wizard better is a low prio for me.

Kingster2 commented 4 years ago

Okay, next time I’ll try the revoke button. Last time it didn’t do what I expected, so the ‘easy’ way was to remove the device.

For now thankx for your work. Kindly appreciated.

erwinvang commented 4 years ago

Now it seems to remember me. I reentered login details. 16D01C40-8C54-4678-86E4-1CD10A301744 ![Uploading BE88BA4D-6127-44AA-8C30-0C6D0EA0C88F.png…]()

However, ring does not seem to activate general alarm on pushing the ring doorbell. Any ideas how to fix it.

erwinvang commented 4 years ago

364E8538-06C3-49D1-B01D-CBEB931D3A7C

kaohlive commented 4 years ago

Not receiving events in generally a enable the event again (using the ring app or the workflow action from Homey). Some people reported that the ring events seem to get disabled at random moments. But that is really hard to debug seems that is an Ring api thing.

So short answer: try restarting the Ring app in Homey OR use a workflow to disable first and then enable the events you wish to register for.

No that I am thinking about it, the app registers the event on init, so if it is disabled in Ring at that moment you might get stuck in such a disabled state, so try to restart the app while your Ring device has the events you want enabled.