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

optional mount path for approle #292

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

Conversation

llafage-claranet
Copy link

Actually the path for an approle is fixed in code:"/v1/auth/approle/login".
This PR add the possibility to support customizable path, for example, "/v1/auth/my_approle/login"

@llafage-claranet llafage-claranet requested a review from a team June 6, 2023 14:49
@hashicorp-cla
Copy link

hashicorp-cla commented Jun 6, 2023

CLA assistant check
All committers have signed the CLA.

@llafage-claranet
Copy link
Author

No test as I don't see custom path testing in spec/integration/api/auth_spec.rb

@llafage-claranet
Copy link
Author

Hello, is it possible to have a review please ? as the change is very small, it should be fast :)

@AnPucel
Copy link
Contributor

AnPucel commented Jun 20, 2023

Hi there! We are definitely happy to take a look at this PR! However, our team has inherited this project and we are currently working on ironing out issues with the CI & automated tests. Once we have those fixed we'll be able to review the PRs and take changes.

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.

3 participants