You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/09/02 20:59:16 UTC

[GitHub] [airflow] alwaysmpe edited a comment on issue #17833: AwsBaseHook isn't using connection.host, using connection.extra.host instead

alwaysmpe edited a comment on issue #17833:
URL: https://github.com/apache/airflow/issues/17833#issuecomment-912050468


   FYI: I'm currently moving house. Should have a look at this over the weekend. My thought was leave the current functionality working as is with a deprecated warning. Then if the host value in extra isn't present, use the connection defined version. Probably needs to combine host/port number, so I'll add a function to the connection to do that.


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org