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

Bring in AAD auth from dbt-synapse? #45

Closed
dataders opened this issue Aug 27, 2020 · 3 comments
Closed

Bring in AAD auth from dbt-synapse? #45

dataders opened this issue Aug 27, 2020 · 3 comments

Comments

@dataders
Copy link
Collaborator

@JosephCDonaldson mentioned they'd like AAD auth available as part of dbt-sqlserver. Perhaps I open a PR to try and merge our connections.py?

@mikaelene
Copy link
Collaborator

That would be great as long as the other auths work. Merge it in 0.18 if you find that it works. I’ll try to release it when dbt 0.18 is out.

@dataders
Copy link
Collaborator Author

We just realized we're missing one key AAD auth method, service principal. We'll aim to have a PR in by the end of the week with the 3 supported AAD methods plus service principal by the end of the week.

@mikaelene
Copy link
Collaborator

This is solved by PR #53 and #55

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

No branches or pull requests

2 participants