-
Notifications
You must be signed in to change notification settings - Fork 59
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
Add browser IDC authentication method #950
Conversation
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the dbt-redshift contributing guide. |
Latest pushed sha re-confirmed builds a model correctly 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few suggestions, nothing blocking. I like the refactor to a more functional approach.
@cla-bot check |
CLA-bot is not working at the moment. The PR author is a dbt Labs employee, hence has signed the CLA. Merging with manual verification. |
partially completes #898
Problem
We want to add additional authentication methods using IAM Identity Center for Redshift.
Solution
This Pull request completes one requirement by adding the browser-based authentication method.
I've refactored the code to be purely functional and assemble parameters in a tail-recursive fashion so the code reflects a gradual build up of reusable chunks.
Along with this PR is a lot of research and experimentation in Redshift/Iam to get a model to build on Redshift using this method!
Checklist