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

[Web UI] Refresh tokens when GraphQL request produces 401 #1976

Merged
merged 2 commits into from
Sep 12, 2018

Conversation

jamesdphillips
Copy link
Contributor

What is this change?

Better handles case where server has invalidated the authorization
token pair or the server's secret has changed.

Why is this change necessary?

Closes #1975

Does your change need a Changelog entry?

undefined

Do you need clarification on anything?

null

Were there any complications while making this change?

my poor debugging skills..?

Have you reviewed and updated the documentation for this change? Is new documentation required?

[]

Better handles case where server has invalidated the authorization
token pair or the server's secret has changed.

Signed-off-by: James Phillips <jamesdphillips@gmail.com>
@jamesdphillips
Copy link
Contributor Author

Maybe related to #2009.

In this way if the server invalidated the access token for whatever
reason we can still try asking for a new access token before having the
user re-authenticate.

Signed-off-by: James Phillips <jamesdphillips@gmail.com>
@jamesdphillips jamesdphillips changed the title [Web UI] Flag authorization tokens when GraphQL request produces 401 [Web UI] Refresh tokens when GraphQL request produces 401 Sep 6, 2018
Copy link
Contributor

@10xjs 10xjs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🌮

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component:web-ui Sensu dashboard improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants