Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

Streaming API Changed #100

Closed
lukiffer opened this issue Oct 18, 2018 · 2 comments
Closed

Streaming API Changed #100

lukiffer opened this issue Oct 18, 2018 · 2 comments

Comments

@lukiffer
Copy link
Collaborator

It looks like all requests to the streaming API (including the initial notify call) are returning a permissions error:

{'data': {'error': '2016', 'message': "We are sorry, but you don't have permission to perform this action.", 'reason': 'No Such Permissions'}, 'success': False}

I'll continue looking into the underlying cause and post any updates here.

@lukiffer
Copy link
Collaborator Author

So after some (very) cursory investigation, it appears that nothing has changed about their API signatures, only that their server-side model validation has gotten pickier about stray properties.

In https://github.com/tchellomello/python-arlo/blob/master/pyarlo/__init__.py#L119 we're simply appending params, but anything that's already on self.__params is preserved. And on https://github.com/tchellomello/python-arlo/blob/master/pyarlo/__init__.py#L86-L87 we're always including username and password fields on the request body.

If we omit these from the body, the request works as expected. I've made this change locally in my homeassistant instance and everything appears to be back to normal. I'll push up to a branch so we can get wider testing.

@tchellomello
Copy link
Owner

Fixed by #101

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

No branches or pull requests

2 participants