alwaysmpe opened a new issue #17833:
URL: https://github.com/apache/airflow/issues/17833


   <!--
   Welcome to Apache Airflow!
   
   Please complete the next sections or the issue will be closed.
   -->
   
   **Apache Airflow version**:
   
   Version: v2.1.2
   Git Version: .release:2.1.2+d25854dd413aa68ea70fb1ade7fe01425f456192
   
   **OS**:
   
   PRETTY_NAME="Debian GNU/Linux 10 (buster)"
   NAME="Debian GNU/Linux"
   VERSION_ID="10"
   VERSION="10 (buster)"
   VERSION_CODENAME=buster
   ID=debian
   HOME_URL="https://www.debian.org/";
   SUPPORT_URL="https://www.debian.org/support";
   BUG_REPORT_URL="https://bugs.debian.org/";
   
   **Apache Airflow Provider versions**:
   
   
   **Deployment**:
   
   Docker compose using reference docker compose from here: 
https://airflow.apache.org/docs/apache-airflow/stable/start/docker.html
   
   **What happened**:
   
   Connection specifications have a host field. This is declared as a member 
variable of the connection class here:
   
https://github.com/apache/airflow/blob/96f7e3fec76a78f49032fbc9a4ee9a5551f38042/airflow/models/connection.py#L101
   It's correctly used in the BaseHook class, eg here:
   
https://github.com/apache/airflow/blob/96f7e3fec76a78f49032fbc9a4ee9a5551f38042/airflow/hooks/base.py#L69
   However in AwsBaseHook, it's expected to be in extra, here:
   
https://github.com/apache/airflow/blob/96f7e3fec76a78f49032fbc9a4ee9a5551f38042/airflow/providers/amazon/aws/hooks/base_aws.py#L404-L406
   
   **What you expected to happen**:
   
   AwsBaseHook should use the connection.host value, not connection.extra.host
   
   **How to reproduce it**:
   
   See above code
   
   **Anything else we need to know**:
   
   
   **Are you willing to submit a PR?**
   
   Probably. It'd also be good for someone inexperienced.
   
   <!---
   This is absolutely not required, but we are happy to guide you in 
contribution process
   especially if you already have a good understanding of how to implement the 
fix.
   Airflow is a community-managed project and we love to bring new contributors 
in.
   Find us in #airflow-how-to-pr on Slack!
    -->
   


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


Reply via email to