Add proxy support for Snowflake Connections

[mansfieldj3]

Description

Ability to specify a proxy host / port / username / password as part of the snowflake connection extra parameters

Use case/motivation

At current a proxy defined using environmental variables in the system can be used by the snowflake hook / operators, however this can have unintended impacts and so it would be better to be able to define it inside the snowflake connection directly

Related issues

No response

Are you willing to submit a PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[boring-cyborg]

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

[Arunodoy18]

HI, I think i can resolve this issue , .take-issue
Thank you.

[suii2210]

Hi! @mansfieldj3
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!

[mansfieldj3]

Thanks for the prompt response @suii2210 and @Arunodoy18, the raised PR addresses my concern so all good on my end

[mansfieldj3]

As previous merge request was closed without being implemented, I'll recreate the prior merge request by @Arunodoy18 updating to fix static errors flagged and any subsequent issues raised