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

Support GitHub (Or Generic) Authentication #346

Open
rfpludwick opened this issue Feb 15, 2023 · 1 comment
Open

Support GitHub (Or Generic) Authentication #346

rfpludwick opened this issue Feb 15, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@rfpludwick
Copy link

SUMMARY

Vault supports GitHub authentication. Please either directly support it in this collection, or allow for a generic authentication method like what HashiCorp have done with Terraform's Vault provider.

ISSUE TYPE
  • Feature Idea
COMPONENT NAME

Unsure...

ADDITIONAL INFORMATION

Some folks have home labs, like me, and are using Vault with GH auth for simplicity's sake.

This could be an approach in allowing generic auth, rather than specifically GitHub. This mimics HashiCorp's approach.

auth_method: generic
auth_path: auth/github/login
auth_params:
  token: abcxyz
@briantist briantist added the enhancement New feature or request label Feb 18, 2023
@briantist
Copy link
Collaborator

Hi @rfpludwick welcome, thanks for submitting this!

I'm especially intrigued by the generic auth idea. That one might be something I'd consider implementing as it would be a little trickier than established concrete methods.
The main reasons are that:

  1. This collection is mostly a wrapper around the hvac library, and it does not itself have such a thing. It does have the Adapter's login method, but this will need to be treated carefully. I am also a maintainer on that library, and I have been trying to work on the Adapter system, so I can say that this way of doing things would be a little unstable right now, until we come up with a future plan for that (I have some ideas, still working with the other maintainers to determine our path forward).
  2. Testing for the generic method will be interesting... it'll have to stray a bit on the integration side from some of the existing tests, but I think we could achieve it by trying it against some existing auth methods so we reuse some of the existing test infra, but that may prove problematic.

In any case, right now the only thing preventing adding a new auth method like GH auth is having someone contribute it!
If you're interested in adding this yourself we have a Contributor guide that can help you get started. Don't hesitate to ask any additional questions as needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants