uranusjr commented on a change in pull request #19789:
URL: https://github.com/apache/airflow/pull/19789#discussion_r755479008
##########
File path: airflow/providers/amazon/aws/hooks/base_aws.py
##########
@@ -539,7 +539,7 @@ def expand_role(self, role: str) -> str:
if "/" in role:
return role
else:
- session, endpoint_url = self._get_credentials()
+ session, endpoint_url = self._get_credentials(region_name=None)
Review comment:
Doesn’t really matter since it’s internal; I like being explicit, so
let’s not. (Actually I would’ve made it a keyword-only argument normally, but
this is already a hot fix and I don’t want to complicate things unnecessarily.)
--
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]
