dabla commented on PR #41400:
URL: https://github.com/apache/airflow/pull/41400#issuecomment-2284387706
> So it seems that the MS Graph operator does not expect an Azure
Connection, but rather an HTTP connection, with extra_dejson `tenantId`. Would
it be ok if I do a check on `conn.conn_type`?
>
> ```
> if conn.conn_type == 'http':
> tenant_id = config.get("tenant_id")
> elif conn.conn_type == 'azure':
> tenant_id = config.get("tenantId")
> ```
>
> My use case is that we use the same credentials (service principals) to
authenticate the Airflow instance, both to the Azure Resource Manager, as well
as to the MS Graph API.
>
> Alternatively, we can raise an exception if the supplied `conn_id` is of
the wrong type?
>
> cc @dabla
We could raise an exception once the MSGraph operator would require an Azure
connection type, something that would be nice once all parameters vould also be
specified in the Azure connection type.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
