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

Use forge passwords #211

Open
hjudt opened this issue May 3, 2022 · 0 comments
Open

Use forge passwords #211

hjudt opened this issue May 3, 2022 · 0 comments

Comments

@hjudt
Copy link
Contributor

hjudt commented May 3, 2022

I am opening this issue as requested in #48 (comment)

Instead of having to configure separate passwords (e.g. in ~/.authinfo.gpg), code-review should be able to use an existing forge configuration to avoid duplicate work for the user. See #48 (comment) for possible solutions.

Preferably, I'd just do without any additional custom option and just use the 'forge key to access the defined password. IMHO, a custom option would only be a good idea for current users of code-review that have set everything up, but that may not be that many at the moment.

Without any further modifications though, only relying on the forge set up will not be enough; code-review requires more entries than forge. So in another step, one should probably make code-review use forge-alist and use fewer entries for the servers (e.g. gitlab.com/api vs gitlab.com/api/v4 vs gitlab.com.

wandersoncferreira pushed a commit that referenced this issue May 3, 2022
code-review searches for authentication data in ~/.authinfo.gpg by looking
for logins ending with "^code-review", like magit forge does with looking
for logins ending with "^forge". Add a customization option to make
code-review look and use those configured for forge, so that users of
forge can use the entries they have configured already.

This fixes issue #211.
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

No branches or pull requests

1 participant