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

ensure note is always unique to allow logins on multiple machines #6

Merged
merged 1 commit into from
Jun 25, 2014
Merged

ensure note is always unique to allow logins on multiple machines #6

merged 1 commit into from
Jun 25, 2014

Conversation

tkellen
Copy link
Contributor

@tkellen tkellen commented Jun 10, 2014

This appends the current date to prevent validation failures due to a token already existing for a given name (try authing the same command on two computers, or nuking your local config without nuking the token in github to see what I mean).

{"message":"Validation Failed","documentation_url":"https://developer.github.com/v3/oauth_authorizations/#create-a-new-authorization","errors":[{"resource":"OauthAccess","code":"already_exists","field":"description"}]}```

@tkellen
Copy link
Contributor Author

tkellen commented Jun 24, 2014

ping

@rvagg
Copy link
Owner

rvagg commented Jun 25, 2014

works for me!

rvagg added a commit that referenced this pull request Jun 25, 2014
ensure note is always unique to allow logins on multiple machines
@rvagg rvagg merged commit 8460363 into rvagg:master Jun 25, 2014
@tkellen
Copy link
Contributor Author

tkellen commented Jun 25, 2014

sweet, thanks!

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.

2 participants