MeltanoLabs / target-snowflake

Singer Target for the Snowflake cloud Data Warehouse
https://hub.meltano.com/loaders/target-snowflake--meltanolabs/
Other
8 stars 18 forks source link

feat: Add support for browser authentication #204

Closed dlouseiro closed 6 days ago

dlouseiro commented 1 month ago

The purpose of this PR is to add support for SSO authentication via external browser (documented here).

As I detailed in this issue, while the most common authentication methods used for production users (Snowflake service account) are basic authentication (user/password) and RSA key-pair authentication, local executions can be done with personal users which, in some companies' Snowflake setup don't have a password, but can only authenticate via SSO (using an external browser).

Implementation details:

Closes https://github.com/MeltanoLabs/target-snowflake/issues/203

dlouseiro commented 1 month ago

FYI, this is my first PR in this repo (or in any other repo from MeltanoLabs so I may have something missing). Would of course appreciate some help from the reviewers to guide me in the right direction!

edgarrmondragon commented 3 weeks ago

@dlouseiro thanks for contributing! Would you mind rebasing your branch to the latest main?

dlouseiro commented 2 weeks ago

@dlouseiro thanks for contributing! Would you mind rebasing your branch to the latest main?

Sure, will do!

dlouseiro commented 2 weeks ago

@dlouseiro thanks for contributing! Would you mind rebasing your branch to the latest main?

Sure, will do!

DOne @edgarrmondragon