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

feat: Add new config parameter private_key #260

Merged
merged 4 commits into from
Sep 19, 2024

Conversation

haleemur
Copy link
Contributor

Description

Add a config parameter private_key. This allows a user of target-snowflake to specify the private_key directly.

This is useful in cases where it is difficult to save the private_key to a file but easy to pass in an environment variable (e.g. a container)

@haleemur haleemur changed the title Config param private key feat: Add new config parameter private_key Sep 19, 2024
@edgarrmondragon edgarrmondragon self-assigned this Sep 19, 2024
Copy link
Member

@edgarrmondragon edgarrmondragon left a comment

Choose a reason for hiding this comment

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

Awesome, thanks @haleemur!

@edgarrmondragon edgarrmondragon merged commit decb72b into MeltanoLabs:main Sep 19, 2024
2 of 9 checks passed
edgarrmondragon added a commit to MeltanoLabs/tap-snowflake that referenced this pull request Sep 20, 2024
# Description
* Copy code over from target-snowflake
MeltanoLabs/target-snowflake#260, now there is
parity between how the target & tap can connect to snowflake (3 ways:
password, key pair & browser authentications)
* This refactor uses `Enum` for `auth_method` for greater type safety.

---------

Co-authored-by: Edgar Ramírez Mondragón <[email protected]>
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