Skip to content

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.

@llafage-claranet
Copy link
Author

Hello, some news maybe ?

@mouchymouchy
Copy link

Hi there
Any new about this PR ?

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.

4 participants