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

Fail loudly with an invalid token #67

Closed
benbalter opened this issue Aug 3, 2016 · 4 comments · Fixed by #75
Closed

Fail loudly with an invalid token #67

benbalter opened this issue Aug 3, 2016 · 4 comments · Fixed by #75

Comments

@benbalter
Copy link
Contributor

No description provided.

@parkr
Copy link
Member

parkr commented Aug 3, 2016

Is there an endpoint we can use to check this?

@benbalter
Copy link
Contributor Author

I was thinking along the lines of not rescuing here. Either:

  1. The user gives us a token, and the token is invalid. They've shown intent to use the token, but we're unable to do so and we should error.
  2. The user gave us a repo, and the repo is invalid. They've shown intent to use a repo's metadata, but we're unable to do so, and should error.

@jekyllbot
Copy link
Contributor

This issue has been automatically marked as stale because it has not been commented on for at least two months.

The resources of the Jekyll team are limited, and so we are asking for your help.

If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in two months if no further activity occurs. Thank you for all your contributions.

@jekyllbot jekyllbot added the stale label Oct 4, 2016
@parkr
Copy link
Member

parkr commented Oct 4, 2016

Lol, @jekyllbot.

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

Successfully merging a pull request may close this issue.

3 participants