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

Ask for a token, not a password #2742

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

smortex
Copy link

@smortex smortex commented Apr 16, 2021

According to this comment, the usage of a password is currently broken but it is possible to pass an access token instead.

This access token is then stored in the configuration file, so adjust the message to ask for an access token, tell the user where then can generate a new one, provide the required scope, and do not tell that the value is not stored.

This would help people who are configuring a new account figure out what is going on more quickly 😉

According to [this comment][ref], the usage of a password is currently
broken but it is possible to pass an access token instead.

This access token is then stored in the configuration file, so adjust
the message to ask for an access token, tell the user where then can
generate a new one, provide the required scope, and do not tell that the
value is not stored.

[ref]: mislav#2655 (comment)
@smortex
Copy link
Author

smortex commented Mar 25, 2022

CI got fixed 💯 I rebased on top of master.

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.

1 participant