ferruzzi commented on a change in pull request #16771:
URL: https://github.com/apache/airflow/pull/16771#discussion_r664196614
##########
File path: airflow/providers/amazon/aws/hooks/base_aws.py
##########
@@ -289,8 +297,8 @@ def _fetch_saml_assertion_using_http_spegno_auth(self,
saml_config: Dict[str, An
raise ValueError('Invalid SAML Assertion')
return saml_assertion
- def _assume_role_with_web_identity(self, role_arn, assume_role_kwargs,
base_session):
- base_session = base_session or botocore.session.get_session()
+ def _get_web_identity_credential_fetcher(self):
Review comment:
I know this didn't have one previously, but could you drop a return type
on the signature? I think it should be `botocore.credentials`?
--
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]
