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

Snowflake Connector Requires Deprecated Region Id #10142

Open
nigelainscoe opened this issue Jun 12, 2019 · 3 comments

Comments

Projects
None yet
3 participants
@nigelainscoe
Copy link

commented Jun 12, 2019

Expected behaviour:
Account name is entered as described in the Account name details section here.
Login is successful

Actual Behaviour:
Account name is entered as described in the Account name details section here.
Login fails with the error "DEBUG metabase.middleware.log :: POST /api/database 400 2 s (0 DB calls) {:valid false, :dbname "net.snowflake.client.jdbc.SnowflakeSQLException: JDBC driver not able to connect to Snowflake. Error code: 390100, Message: Incorrect username or password was specified..""

Problem:
Region Id is now deprecated in Snowflake JDBC connections as per the Snowflake documentation

The Snowflake database connector dialog asks for a separate Region Id field. In order to successfully connect, account name and region Id need to be entered in the deprecated manner.

Solution:
At the very least the dialogue should allow successful connection when account name is entered as per the Account name details section here
Better still the deprecated Region ID should be removed and users guided to use the Snowflake account name guidance.

Note
The code in the driver appears to try to solve it with this:

(let [host (if regionid (str account "." regionid) account)]

but using the driver dialog, it doesn't seem to work.
Unfortunately my Clojure is not up to debugging it - although I wondered if the display-name: Region ID from the YAML file is getting used when nothing is entered in that field?

@camsaul

This comment has been minimized.

Copy link
Member

commented Jun 13, 2019

Hi @nigelainscoe what version of Metabase are you using

@flamber

This comment has been minimized.

Copy link
Contributor

commented Jun 13, 2019

@nigelainscoe

This comment has been minimized.

Copy link
Author

commented Jun 14, 2019

@camsaul Version 0.32.8 is correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.