search

snowflakedb / snowflake-connector-net

Snowflake Connector for .NET
Apache License 2.0
179 stars 138 forks source link

Connection caching for SSO and MFA #380

Open vrrx opened 3 years ago

vrrx commented 3 years ago

Are there any plans on implementing connection/token caching for SSO and MFA authentication, which would reduce the number of prompts when opening connection multiple times? Is this on your roadmap?

Based on this info, implementation for this already exists for ODBC/JDBC drivers and Python connector.

sfc-gh-dszmolka commented 1 year ago

hi and thank you for submitting this request. apologies for the long period without response. sadly, the request is still relevant even today - driver still not has this capability to my knowledge. at least the request is already known to the product team, so hopefully it will get prioritised and started in the near future. will update this issue once i have any news.

sfc-gh-dszmolka commented 5 months ago

PR in progress https://github.com/snowflakedb/snowflake-connector-net/pull/921

markjefford commented 2 months ago

Hi there. I'm hoping this feature would prevent the browser popping up every time I open a connection, and so have cloned the repo to test the branch locally.

I have set ALLOW_SSO_TOKEN_CACHING=true in the connection string, and also run "ALTER ACCOUNT SET ALLOW_ID_TOKEN = TRUE".

I am still not seeing caching take place. Am I missing something? Plus, is there a timeline for a merge?

Thanks and Regards Mark

sfc-gh-dszmolka commented 2 months ago

you're not missing anything @markjefford ; until the PR is merged (and, a new driver is released on top of the merged code, and that new driver release is installed on your environment), this feature is not available.

I'll keep this ticket posted (actually, any ticket posted..) when any new information is available. At this moment, unfortunately no foreseeable estimation for the merge and which driver release could carry the new feature.

gdoron commented 2 months ago

Would also appreciate this feature so my fellow devs won't stick a bomb under my car every time they need to develop snowflake stuff.

sfc-gh-dszmolka commented 1 month ago

short update: dev team needs to address some concerns raised internally during security review of the new feature + all our resources for this library are currently focused on fixing the memory leak issue (1004). Apologies this takes so long and thank you for bearing with us while this new capability is fully implemented and released. I'll keep this thread posted with new information, as they become available to me.