Skip to content

db/snowflake: clarify connection_url format when using keypair authentication #940

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Sep 22, 2025
Merged

db/snowflake: clarify connection_url format when using keypair authentication #940

merged 5 commits into from
Sep 22, 2025

Conversation

vinay-gopalan
Copy link
Contributor

Makes documentation explicitly clear on how to format the connection_url when using keypair authentication. Mainly explicitly calling out the following:

  • users should not prepend or include any username:password information (templated or hard-coded) in the connection URL submitted to Vault
  • users can supply optional query parameters if they wish to do so

Other changes:

  • Adds missing documentation from 1.18 and 1.16 branches. While the feature was backported to those versions, looks like the documentation was not backported to those branches at the time.

@vinay-gopalan vinay-gopalan requested a review from a team as a code owner September 12, 2025 18:17
@github-actions github-actions bot added the Vault Content update for Vault product docs label Sep 12, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 12, 2025 •
edited
Loading

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Mon Sep 22 18:46:19 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Mon Sep 22 18:39:17 UTC 2025

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 12, 2025 •
edited
Loading

Broken Link Checker

No broken links found! 🎉

schavis
schavis requested changes Sep 12, 2025
View reviewed changes
@cbiguet cbiguet added the documentation Improvements or additions to documentation label Sep 18, 2025
schavis
schavis requested changes Sep 22, 2025
View reviewed changes
@vinay-gopalan vinay-gopalan merged commit ce19ddd into vault/202509 Sep 22, 2025
15 checks passed
@vinay-gopalan vinay-gopalan deleted the vinayg/snowflake-keypair-query-params branch September 22, 2025 20:38
@vinay-gopalan vinay-gopalan mentioned this pull request Sep 23, 2025
Merged
vinay-gopalan added a commit that referenced this pull request Sep 23, 2025
@vinay-gopalan
db/snowflake: clarify connection_url format when using keypair auth…
d2e5931 
…entication in 1.21 documentation (#985)

Makes same changes as #940 but for the 1.21 release.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cbiguet cbiguet cbiguet left review comments

@schavis schavis schavis approved these changes

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation Vault Content update for Vault product docs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vinay-gopalan @cbiguet @schavis