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

#45 - auth and refresh token are same for non-admins #64

Merged
merged 1 commit into from
Feb 13, 2020

Conversation

jasonbahl
Copy link
Collaborator

  • if the token doesn't belong to the current user AND the current user doesn't have proper caps, throw an error

closes #45

- if the token doesn't belong to the current user AND the current user doesn't have proper caps, throw an error
@jasonbahl jasonbahl self-assigned this Jan 22, 2020
@jasonbahl jasonbahl closed this Jan 29, 2020
@jasonbahl jasonbahl deleted the bug/#45-tokens-identical-for-non-admins branch January 29, 2020 14:49
@trevorblades
Copy link

Hey @jasonbahl, just curious why you ended up closing this. Does it not fix the issue referenced in the description?

@henrikwirth
Copy link
Collaborator

Oh yeah, I think this has not been merged. @jasonbahl was that intended?

@abetoots
Copy link

Hey @jasonbahl , just bumping this as I am curious as well.

@jasonbahl jasonbahl restored the bug/#45-tokens-identical-for-non-admins branch February 13, 2020 22:48
@jasonbahl jasonbahl reopened this Feb 13, 2020
@jasonbahl
Copy link
Collaborator Author

My bad! I was trying to clean up branches and got a little trigger happy. 🤦‍♂

@jasonbahl jasonbahl merged commit d77da3f into master Feb 13, 2020
@henrikwirth henrikwirth deleted the bug/#45-tokens-identical-for-non-admins branch February 13, 2020 22:54
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

Successfully merging this pull request may close these issues.

4 participants