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

trap failed authentication #41

Merged
merged 1 commit into from
Jul 1, 2016
Merged

trap failed authentication #41

merged 1 commit into from
Jul 1, 2016

Conversation

jgravois
Copy link
Collaborator

@jgravois jgravois commented Jul 1, 2016

about half a dozen times i've found myself in a situation where GitHub rejects the token that is passed when attempting to tag a release. this PR just traps the problem and points the user in the right direction to clear their existing config file.

i haven't been able to determine the root cause, and i don't know if its likely to occur again now that you've moved to using ghauth but if i'm not the only one it would be a lot cooler for gh-release to trigger a fresh prompt to allow the user to authenticate. i just couldn't figure out how to do it.

@bcomnes
Copy link
Collaborator

bcomnes commented Jul 1, 2016

lgtm

@ungoldman
Copy link
Owner

thanks! merging

@ungoldman ungoldman merged commit a970520 into ungoldman:master Jul 1, 2016
@jgravois jgravois deleted the clear-cache branch May 2, 2017 16:27
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.

None yet

3 participants