Skip to content

Conversation

@iamcxa
Copy link
Contributor

@iamcxa iamcxa commented Oct 22, 2025

Summary

This PR adds support documentation for Snowflake Key Pair Authentication in addition to the existing password-based authentication method.

Changes

  • connect-to-warehouse.md: Added comprehensive documentation for Snowflake Key Pair authentication
    • Reorganized Snowflake section to clarify two authentication methods are supported
    • Split field tables into: Common Fields, User & Password Authentication, and Key Pair Authentication
    • Added new fields: private_key, private_key_passphrase
    • Included reference link to Snowflake's official key pair authentication documentation
    • Improved table formatting for better readability (also applied to Databricks section)

Screenshots

image

Test Plan

  • Documentation structure is clear and well-organized
  • All field descriptions are accurate and complete
  • External link to Snowflake docs is valid
  • Markdown formatting is correct

Impact

  • Documentation: Users can now find information about both password-based and key pair authentication for Snowflake
  • User Experience: Clearer organization makes it easier to understand authentication options
  • Security: Promotes the use of more secure key pair authentication method

Related: DRC-1952

@iamcxa iamcxa requested a review from popcornylu October 22, 2025 04:35

### Snowflake
We only support password-based authentication.
We support two authentication methods for Snowflake:
Copy link
Contributor

Choose a reason for hiding this comment

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

Add additional line here.

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a new-line

image

@popcornylu popcornylu self-requested a review October 22, 2025 07:09
Copy link
Contributor

@popcornylu popcornylu left a comment

Choose a reason for hiding this comment

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

LGTM

@popcornylu popcornylu merged commit 5d236f2 into main Oct 22, 2025
@popcornylu popcornylu deleted the feature/drc-1952-doc-support-snowflake-keypair branch October 22, 2025 07:10
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.

3 participants