tsonic04 opened a new pull request, #54329:
URL: https://github.com/apache/airflow/pull/54329

   **Title:**
   Fix SAS token handling in BlobServiceClient connection logic and a minor 
clarity edit on oracle.py
   
   **Description:**
   This PR fixes the logic for constructing the account_url and credential 
parameters when using a SAS token to create BlobServiceClient and 
AsyncBlobServiceClient instances.
   Previously, the method would append the SAS as a path onto the account url. 
The path error would cause failures of connections if the SAS token did not 
include 'https'. Most minor correction would have been to swap the "/" for a 
"?" but providing the token to credential is more proper. 
   
   Oracle.py change was to correct a flynt warning that got raised during 
pre-commit check. Switched out f-string for the .format logic and added some 
variables for clarity btu otherwise code is the same.  


-- 
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]

Reply via email to