potiuk edited a comment on issue #20457:
URL: https://github.com/apache/airflow/issues/20457#issuecomment-999190957


   > More deeply though, `expand_role()` used to depend on boto’s 
`get_client_type()` with a dynamic client type, which is deprecated in newer 
Boto versions. Tt now uses `_get_credentials()` instead to avoid the 
deprecation. But `expand_role()` does not tak a region argument and therefore 
cannot pass one to `_get_credentials()`. If we keep compatibility on 
`_get_credentials()`, we’d break compatibility on `expand_role()` instead.
   
   Can't we pass `None` explicitly as `region` from the `expand_role()`  and 
leave the `region` argument Optional but without default ?
   


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