SNOW-1855886 gracefully ignore when default region us-west-2 is used in DSN or NewConnector #1278
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
us-west-2
region is the default region in Snowflake, and a special one. We already had checks for the case where someone accidentally configures it inRegion
, but forAccount
we did not have any, which led to anAccount
(mis)configured asmyaccount.us-west-2
instead of omitting the region (as is the default) and usingmyaccount
, not being able to connect to Snowflake and instead failing with HTTP404.This change attempts to step over such simple misconfiguration gracefully and
.us-west-2
from Account instead of allowing the connection to proceed, which then surely will fail on the Snowflake backendChecklist