mik-laj commented on a change in pull request #17448: URL: https://github.com/apache/airflow/pull/17448#discussion_r683842856
########## File path: airflow/providers/amazon/aws/secrets/secrets_manager.py ########## @@ -96,28 +97,74 @@ def __init__( @cached_property def client(self): - """Create a Secrets Manager client""" + """ + Create a Secrets Manager client + """ session = boto3.session.Session( - profile_name=self.profile_name, + profile_name=self.profile_name ) return session.client(service_name="secretsmanager", **self.kwargs) - def get_conn_uri(self, conn_id: str) -> Optional[str]: + def _get_extra(self, secret, conn_string): + if 'extra' in secret: + extra_dict = secret['extra'] + kvs = "&".join([f"{key}={value}" for key, value in extra_dict.items()]) Review comment: What do you think about [urllib.parse.urlencode](https://docs.python.org/3/library/urllib.parse.html#urllib.parse.urlencode). I'm afraid the current implementation may have problems with some special characters. -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org