-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Either the 2FA documentation is wrong or the client is broken #592
Comments
You'll need to pass the OTP on every request. Since an OTP is no longer valid after it's used, it's not feasible for a loop like this. Instead, I recommend creating a Personal Access Token and use that as your password. With PATs, you won't need to worry about sending the OTP. |
Hmm, does that explain why the OTP is also in the request GET query string? On Sat, May 23, 2015 at 8:36 PM, Wynn Netherland notifications@github.com
Kurtis Rainbolt-Greene, Hacker |
A second look at the request headers indicates they're being sent, they're just also getting sent in the query because we're doing a We should fix that up. Patches are always welcome. |
The text was updated successfully, but these errors were encountered: