suii2210 commented on issue #59078: URL: https://github.com/apache/airflow/issues/59078#issuecomment-3622338862
Hi! I’d like to take this issue. I’m proposing to add proxy support to the Snowflake connection by allowing the following keys inside the connection extra JSON: proxy_host proxy_port proxy_user proxy_password My plan: Update SnowflakeHook.get_conn to read these extra fields and pass them as keyword arguments to snowflake.connector.connect(). Ensure proxy_port is validated and cast to int. If invalid, raise an AirflowException. Extras will override environment proxy variables (explicit > implicit), while keeping existing behavior unchanged if no extras are provided. Add unit tests to confirm: proxy extras are passed correctly invalid port raises a clear error Update Snowflake provider docs with an example of the extra JSON format. This will keep the feature backwards-compatible and make proxy configuration explicit per-connection. If this approach looks good, I’ll open a PR following the Airflow guidelines. Thanks! -- 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]
