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

auth_login: enforce empty client token #2029

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

benashz
Copy link
Contributor

@benashz benashz commented Sep 26, 2023

Fixes a failure when an auth_login is specified and a vault token is picked up from the runtime/execution environment. In the meantime the workaround is to ensure that VAULT_TOKEN is unset, and there exists no vault token on disk e.g: ~/.vault-token

other fixes:

  • parallelize AuthLogin tests
  • ensure VAULT_TOKEN is not set from the test make targets

Relates to hashicorp/vault#23216 #2026

  • add vault token test to all AuthLogin.Login() tests

@benashz benashz added this to the 3.21.0 milestone Sep 26, 2023
@benashz benashz force-pushed the VAULT-20288/auth-login-expunge-env-client-token branch 2 times, most recently from 7f794f2 to 42c3a97 Compare September 26, 2023 20:01
@alex-ikse
Copy link

Tested and approved with my use case described in my closed PR #2026 : excited to see release 3.21.0.
Thank you for your consideration.

other fixes:
- parallelize AuthLogin tests
- ensure VAULT_TOKEN is not set from the test make targets
@benashz benashz force-pushed the VAULT-20288/auth-login-expunge-env-client-token branch from 42c3a97 to a0b9d2e Compare September 26, 2023 20:36
@benashz benashz changed the title WIP: auth_login: enforce empty client token auth_login: enforce empty client token Sep 26, 2023
@github-actions github-actions bot added size/XL and removed size/M labels Sep 26, 2023
@benashz benashz marked this pull request as ready for review September 26, 2023 20:38
@benashz benashz merged commit c0015bd into main Sep 28, 2023
@benashz benashz deleted the VAULT-20288/auth-login-expunge-env-client-token branch September 28, 2023 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants