Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Managed Identity/native auth #21

Open
svenkerud opened this issue Dec 1, 2023 · 0 comments
Open

Managed Identity/native auth #21

svenkerud opened this issue Dec 1, 2023 · 0 comments

Comments

@svenkerud
Copy link

Hi,

This tap looks very promising, and we are looking to deploy it in our system.
However we were wondering if it would be possible to set up authentication with Managed Identity or using native Auth.
As we avoid the use of service principals where possible.

We think it should only require a relative minor change to this method:
https://github.com/Slalom-Consulting/tap-ms-graph/blob/main/tap_ms_graph/auth.py

We are happy to provide a PR with this change, but would like to discuss the design principles regarding how configuration would look with multiple auth options.

Some of our thoughts are:
add use DefaultAzureCredentials = true as an option
add managed Identity ID as an option

We might also need to think about precedence in the case that multiple authentication methods are configured. As well as configuration validation.

Tagging colleague: @hholgersen

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant