Skip to content

feat: use custom dialer in snowflake connections#5611

Merged
mhilton merged 5 commits intomasterfrom
mhlton/snowflake-network-dialer
Mar 18, 2026
Merged

feat: use custom dialer in snowflake connections#5611
mhilton merged 5 commits intomasterfrom
mhlton/snowflake-network-dialer

Conversation

@mhilton
Copy link
Contributor

@mhilton mhilton commented Mar 13, 2026

When connecting to a snowflake database use an http.Transport defived from the provided dependencies.

Closes: influxdata/idpe#19466

Checklist

Dear Author 👋, the following checks should be completed (or explicitly dismissed) before merging.

  • ✏️ Write a PR description, regardless of triviality, to include the value of this PR
  • 🔗 Reference related issues
  • 🏃 Test cases are included to exercise the new code
  • 🧪 If new packages are being introduced to stdlib, link to Working Group discussion notes and ensure it lands under experimental/
  • 📖 If language features are changing, ensure docs/Spec.md has been updated

Dear Reviewer(s) 👋, you are responsible (among others) for ensuring the completeness and quality of the above before approval.

@mhilton mhilton requested a review from jdstrand March 13, 2026 13:03
@mhilton mhilton requested a review from a team as a code owner March 13, 2026 13:03
mhilton added 5 commits March 18, 2026 16:02
When connecting to a snowflake database use an http.Transport defived
from the provided dependencies.
Update to a newer version of the gosnowflake dependency.
@mhilton mhilton force-pushed the mhlton/snowflake-network-dialer branch from 12109a3 to fd36c6b Compare March 18, 2026 16:11
@mhilton mhilton merged commit a1bdac6 into master Mar 18, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants