dabla commented on PR #61199: URL: https://github.com/apache/airflow/pull/61199#issuecomment-3824976285
> *closes: #61197 > > To be honest, this PR is only a draft. From the Code I can see that **Authority** can be retrieved from connection as an extra parameter. However, there is no information about it in the [documentation](https://airflow.apache.org/docs/apache-airflow-providers-microsoft-azure/stable/connections/powerbi.html#configuring-the-connection). > > So, this is a little bit confusing when using the proxy. So I propose to update the logic of a function _to_msal_proxies_. Return proxies attribute even when the variable authority is not filled. > > **Testing** In our environment I did a functional logic test with and without proxy. It works as expected msal_proxies is only supposed to return proxies if an authority is defined, otherwise I don't think it's supposed to return any proxies. -- 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]
