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

Make resolve-private-key include private-key-options in database details #41941

Merged
merged 3 commits into from Apr 29, 2024

Conversation

lbrdnk
Copy link
Contributor

@lbrdnk lbrdnk commented Apr 29, 2024

Closes #41852

When hosting feature is enabled, database edit GUI does not send private-key-options as part of request. secret/get-secrect-string depends on that value to determine if value should base64 decoded. This PR ensures the value of the key is set correctly in resolve-private-key.

Decode the value by default to match behavior of `file-upload-props`.
@lbrdnk lbrdnk added the .Team/QueryProcessor :hammer_and_wrench: label Apr 29, 2024
@lbrdnk lbrdnk self-assigned this Apr 29, 2024
@lbrdnk lbrdnk requested a review from camsaul as a code owner April 29, 2024 09:34
@lbrdnk lbrdnk added the backport Automatically create PR on current release branch on merge label Apr 29, 2024
Copy link

replay-io bot commented Apr 29, 2024

Status Complete ↗︎
Commit 42852c8
Results
⚠️ 12 Flaky
2424 Passed

@lbrdnk lbrdnk requested a review from a team April 29, 2024 15:50
@lbrdnk lbrdnk added this to the 0.49.8 milestone Apr 29, 2024
@lbrdnk lbrdnk changed the title Match the default behavior of file-upload-props in get-secrect-string Make resolve-private-key include private-key-options set in database details Apr 29, 2024
@lbrdnk lbrdnk removed this from the 0.49.8 milestone Apr 29, 2024
@lbrdnk lbrdnk changed the title Make resolve-private-key include private-key-options set in database details Make resolve-private-key include private-key-options in database details Apr 29, 2024
@lbrdnk lbrdnk merged commit 6581a28 into master Apr 29, 2024
109 checks passed
@lbrdnk lbrdnk deleted the snowflake-rsa-41852 branch April 29, 2024 21:30
github-actions bot pushed a commit that referenced this pull request Apr 29, 2024
…details (#41941)

* Update resolve-private-key and add test
metabase-bot bot added a commit that referenced this pull request Apr 29, 2024
…details (#41941) (#42006)

* Update resolve-private-key and add test

Co-authored-by: lbrdnk <lbrdnk@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Automatically create PR on current release branch on merge .Team/QueryProcessor :hammer_and_wrench:
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issues connecting to Snowflake using RSA Keypair Authentication
2 participants