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

csi: Fix parsing of '=' in secrets at command line and HTTP #15670

Merged
merged 2 commits into from
Jan 3, 2023

Conversation

tgross
Copy link
Member

@tgross tgross commented Jan 3, 2023

Fixes #15663

The command line flag parsing and the HTTP header parsing for CSI secrets incorrectly split at more than one '=' rune, making it impossible to use secrets that included that rune.

The command line flag parsing and the HTTP header parsing for CSI secrets
incorrectly split at more than one '=' rune, making it impossible to use secrets
that included that rune.
@tgross tgross force-pushed the b-csi-volume-secrets-parsing branch from d72b85a to c4cde59 Compare January 3, 2023 19:50
@tgross tgross added this to the 1.4.4 milestone Jan 3, 2023
@tgross tgross marked this pull request as ready for review January 3, 2023 20:32
@tgross tgross added backport/1.2.x backport to 1.1.x release line backport/1.3.x backport to 1.3.x release line backport/1.4.x backport to 1.4.x release line labels Jan 3, 2023
Copy link
Contributor

@angrycub angrycub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a suggestion about an alternative implementation, but nothing that prevents this from going as is.

@vercel
Copy link

vercel bot commented Jan 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
nomad-storybook-and-ui 🔄 Building (Inspect) Jan 3, 2023 at 9:00PM (UTC)

@tgross tgross merged commit e23b3a3 into main Jan 3, 2023
@tgross tgross deleted the b-csi-volume-secrets-parsing branch January 3, 2023 21:28
tgross added a commit that referenced this pull request Jan 3, 2023
The command line flag parsing and the HTTP header parsing for CSI secrets
incorrectly split at more than one '=' rune, making it impossible to use secrets
that included that rune.
tgross added a commit that referenced this pull request Jan 3, 2023
…15673)

The command line flag parsing and the HTTP header parsing for CSI secrets
incorrectly split at more than one '=' rune, making it impossible to use secrets
that included that rune.

Co-authored-by: Tim Gross <tgross@hashicorp.com>
philrenaud pushed a commit that referenced this pull request Jan 23, 2023
The command line flag parsing and the HTTP header parsing for CSI secrets
incorrectly split at more than one '=' rune, making it impossible to use secrets
that included that rune.
@github-actions
Copy link

github-actions bot commented May 5, 2023

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/1.2.x backport to 1.1.x release line backport/1.3.x backport to 1.3.x release line backport/1.4.x backport to 1.4.x release line theme/cli type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to delete external volume because of special character in secret key/value pair
3 participants