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 21:20:05 UTC

[GitHub] [airflow] ashb commented on issue #17833: AwsBaseHook isn't using connection.host, using connection.extra.host instead

ashb commented on issue #17833:
URL: https://github.com/apache/airflow/issues/17833#issuecomment-912065820


   > 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.
   
   That sounds about right, yes


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