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

Prevent Git Gateway users with invalid tokens from logging in. #1209

Merged
merged 2 commits into from
Mar 28, 2018

Conversation

tech4him1
Copy link
Contributor

- Summary

Fixes #647. Also addresses the case where a user changes their GH repo from public to private after Git Gateway is set up.

- Test plan

Manually tested.

- Description for the changelog

Prevent Git Gateway users with invalid tokens from logging in.

- A picture of a cute animal (not mandatory but encouraged)

This often happens when a user changes a repo from public to private, so
we want to make that specific case very clear.
@verythorough
Copy link
Contributor

Deploy preview for netlify-cms-www ready!

Built with commit 6e50b95

https://deploy-preview-1209--netlify-cms-www.netlify.com

@verythorough
Copy link
Contributor

Deploy preview for cms-demo ready!

Built with commit 6e50b95

https://deploy-preview-1209--cms-demo.netlify.com

@tech4him1
Copy link
Contributor Author

@erquhart This is ready to review/merge if you want it in the release.

Copy link
Contributor

@erquhart erquhart left a comment

Choose a reason for hiding this comment

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

Awesome

@erquhart erquhart merged commit 085c88e into master Mar 28, 2018
@erquhart erquhart deleted the 647-git-gateway-auth-check branch March 28, 2018 18:08
brianlmacdonald pushed a commit to brianlmacdonald/netlify-cms that referenced this pull request May 23, 2018
…org#1209)

* Prevent Git Gateway users without permission from login.

* Handle Git Gateway token expiry explicitly.

This often happens when a user changes a repo from public to private, so
we want to make that specific case very clear.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Identity/Git Gateway users with insufficient permissions see UI with "no entries"
3 participants