Skip to content

KeyError in Client #32

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

Closed
ggravlingen opened this issue Mar 25, 2022 · 5 comments
Closed

KeyError in Client #32

ggravlingen opened this issue Mar 25, 2022 · 5 comments
Assignees

Comments

@ggravlingen
Copy link

Hi,

I'm getting this error after updating to 1.1.1. Not sure if this is a local issue or a bug? Noted some changes where made in nearby code in this PR: #29

Error in Microsoft login hook error Traceback (most recent call last):
  File "/workspaces/.../.../integrations/azure/aad/auth_hook.py", line 43, in handle_login
    user = client.users.get_me()
  File "/usr/src/.venv/lib/python3.8/site-packages/microsoftgraph/decorators.py", line 11, in helper
    return func(*args, **kwargs)
  File "/usr/src/.venv/lib/python3.8/site-packages/microsoftgraph/users.py", line 32, in get_me
    return self._client._get(self._client.base_url + "me", params=params)
  File "/usr/src/.venv/lib/python3.8/site-packages/microsoftgraph/client.py", line 227, in _get
    return self._paginate_response(response)
  File "/usr/src/.venv/lib/python3.8/site-packages/microsoftgraph/client.py", line 215, in _paginate_response
    data = list(response.data["value"])
KeyError: 'value'
@Free-devloper
Copy link

Me too Anyone resolved it yet?

@ingmferrer ingmferrer self-assigned this Mar 27, 2022
@ingmferrer
Copy link
Contributor

Release 1.1.2 should resolve this, please upgrade and let me know if everything works as expected.

@ggravlingen
Copy link
Author

Unfortunately not, I'm getting the same error. The error is on line 215:

data = list(response.data["value"])

@ingmferrer
Copy link
Contributor

I forgot to mention that shortly after I commented this issue, I released the 1.1.3 version, so please upgrade to that version because it includes new validations.

@ggravlingen
Copy link
Author

There are no errors now. Thank you!

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

No branches or pull requests

3 participants