Skip to content

fix(snowflake): use disable_ocsp_checks connector arg - #1995

Open
RoyalTS wants to merge 1 commit into
dbt-labs:mainfrom
RoyalTS:fix-snowflake-disable-ocsp-checks
Open

fix(snowflake): use disable_ocsp_checks connector arg#1995
RoyalTS wants to merge 1 commit into
dbt-labs:mainfrom
RoyalTS:fix-snowflake-disable-ocsp-checks

Conversation

@RoyalTS

@RoyalTS RoyalTS commented Jun 1, 2026

Copy link
Copy Markdown

Summary

Test plan

  • uvx hatch run unit-tests tests/unit/test_connections.py tests/unit/test_snowflake_adapter.py
  • uvx pre-commit run --files dbt-snowflake/src/dbt/adapters/snowflake/connections.py dbt-snowflake/tests/unit/test_snowflake_adapter.py dbt-snowflake/tests/unit/test_connections.py dbt-snowflake/.changes/unreleased/Fixes-20260601-154900.yaml

Made with Cursor

Avoid passing the deprecated insecure_mode argument to the Snowflake Python connector while preserving dbt profile compatibility.

Co-authored-by: Cursor <cursoragent@cursor.com>
@RoyalTS
RoyalTS requested a review from a team as a code owner June 1, 2026 15:04
@cla-bot

cla-bot Bot commented Jun 1, 2026

Copy link
Copy Markdown

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @RoyalTS

@github-actions github-actions Bot added the community A PR, or an issue with a PR, from a community member label Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community A PR, or an issue with a PR, from a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant