You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "hussein-awala (via GitHub)" <gi...@apache.org> on 2023/09/24 23:22:56 UTC
[GitHub] [airflow] hussein-awala opened a new pull request, #34590: Use aiohttp.BasicAuth instead of HTTPBasicAuth for aiohttp session in databricks hook
hussein-awala opened a new pull request, #34590:
URL: https://github.com/apache/airflow/pull/34590
closes: #34450
Databricks hook uses aiohttp client session for async methods, but in the method `_a_get_sp_token`, we use `HTTPBasicAuth` instance to provide basic authentication credentials which is not compatible with aiohttp sessions. This PR fixes the issue by providing the credentials using `aiohttp.BasicAuth` instance as we do in the other methods.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] phanikumv merged pull request #34590: Use aiohttp.BasicAuth instead of HTTPBasicAuth for aiohttp session in databricks hook
Posted by "phanikumv (via GitHub)" <gi...@apache.org>.
phanikumv merged PR #34590:
URL: https://github.com/apache/airflow/pull/34590
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Use aiohttp.BasicAuth instead of HTTPBasicAuth for aiohttp session in databricks hook [airflow]
Posted by "rinzool (via GitHub)" <gi...@apache.org>.
rinzool commented on PR #34590:
URL: https://github.com/apache/airflow/pull/34590#issuecomment-1749097717
Hi @hussein-awala, thanks for the fix of https://github.com/apache/airflow/issues/34450
Do you know where a new version of the databricks provider will be deployed with this fix? :pray:
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org