Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue due to removal of region from Snowflake URI #102

Closed
tatiana opened this issue Feb 11, 2022 · 1 comment · Fixed by #95
Closed

Issue due to removal of region from Snowflake URI #102

tatiana opened this issue Feb 11, 2022 · 1 comment · Fixed by #95
Labels
bug Something isn't working priority/critical Critical priority
Milestone

Comments

@tatiana
Copy link
Collaborator

tatiana commented Feb 11, 2022

Version: astro==0.4.0

Context
The way we are building Snowflake URIs seems to be prone to errors.

Recently, a user had reported that Snowflake had deprecated the usage of the region in the host URI, and we made this change:
541038c#diff-4498bebc277444fc4d9662514a3a2e2eacdcdfd139e160abd6fdb16b0e43a854L31

Although the user was happy with the change, other users who were being able to successfully use Astro started experiencing the error:

snowflake.connector.errors.ForbiddenError: 250001 (08001): None: Failed to connect to DB. Verify the account name is correct: GP123.snowflakecomputing.com:443. 000403: 403: HTTP 403: Forbidden

The goal of this ticket is to resolve the issue so both users are able to use Astro, if possible removing any complexity within Astro.

Additional details

Based on the Snowflake documentation:

For example, if your account locator is xy12345:

    If the account is located in the AWS US West (Oregon) region, no additional segments are required and the URL would be xy12345.snowflakecomputing.com.

    If the account is located in the AWS US East (Ohio) region, additional segments are required and the URL would be xy12345.us-east-2.aws.snowflakecomputing.com.

One approach we can consider taking is to remove altogether the need for the TempSnowflakeHook class and trust the user to set the Snowflake host which works for them.

@tatiana tatiana added bug Something isn't working priority/critical Critical priority labels Feb 11, 2022
@tatiana tatiana changed the title Issue due to removal of Snowflake region Issue due to removal of region from Snowflake URI Feb 11, 2022
@tatiana
Copy link
Collaborator Author

tatiana commented Feb 21, 2022

This was solved as part of #95

@tatiana tatiana added this to the 0.6.0 milestone Feb 21, 2022
@tatiana tatiana closed this as completed Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority/critical Critical priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant