-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
CLI: add new -header option to be able to add headers to all cli http request #12508
Conversation
b67ec8c
to
efde361
Compare
efde361
to
e09eb9f
Compare
Hi @ylorenzati, this is an interesting concept. In addition to your changes, we will want to prevent users from providing headers with the prefix It also appears that there are conflicts in |
e09eb9f
to
b50fb08
Compare
Hi @ccapurso, Thank you very much for your time and your feedback. I will implement what you asked as soon as possible (Before the end of the week I hope) |
Sounds good, take your time :). |
b50fb08
to
10e21d3
Compare
10e21d3
to
d963cdf
Compare
Thank you for the |
This PR add a new option to the vault cli: -header with can be specified multiple times. All the parameters specified like that will be added to HTTP headers for all requests.
For example, it will enable us to achieve zero trust with Cloudflare access and still be able to use vault cli
This fix #8754