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.

Sign up for GitHub

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

Jump to bottom

Connection caching for SSO and MFA #380

Open
vrrx opened this issue Sep 30, 2021 · 6 comments
Open

Connection caching for SSO and MFA #380

vrrx opened this issue Sep 30, 2021 · 6 comments
Assignees
@sfc-gh-anugupta
Labels
enhancement The issue is a request for improvement or a new feature parity this feature is supported in other drivers status-pr_pending_merge A PR is made and is under review status-triage_done Initial triage done, will be further handled by the driver team

Comments

@vrrx
Copy link

vrrx commented Sep 30, 2021

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-hchaturvedi sfc-gh-hchaturvedi added the enhancement The issue is a request for improvement or a new feature label Oct 14, 2021
@sfc-gh-jfan sfc-gh-jfan reopened this Jul 1, 2022
@github-actions github-actions bot closed this as completed Jul 2, 2022
@sfc-gh-jfan sfc-gh-jfan reopened this Jul 6, 2022
@sfc-gh-dszmolka sfc-gh-dszmolka added status-triage Issue is under initial triage parity this feature is supported in other drivers labels Jun 7, 2023
@sfc-gh-dszmolka
Copy link
Contributor

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 sfc-gh-dszmolka added status-in_progress Issue is worked on by the driver team and removed status-triage Issue is under initial triage labels Jun 7, 2023
@sfc-gh-dszmolka sfc-gh-dszmolka changed the title Connection caching for SSO Connection caching for SSO and MFA Jun 7, 2023
@sfc-gh-dszmolka sfc-gh-dszmolka mentioned this issue Jun 7, 2023
Closed
@sfc-gh-dszmolka sfc-gh-dszmolka added status-triage_done Initial triage done, will be further handled by the driver team and removed status-in_progress Issue is worked on by the driver team labels Feb 11, 2024
@sfc-gh-dszmolka
Copy link
Contributor

PR in progress #921

@sfc-gh-dszmolka sfc-gh-dszmolka added the status-pr_pending_merge A PR is made and is under review label Jun 7, 2024
@markjefford
Copy link

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
Copy link
Contributor

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
Copy link

gdoron commented Aug 29, 2024

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
Copy link
Contributor

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.

@sfc-gh-dszmolka sfc-gh-dszmolka mentioned this issue Oct 28, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sfc-gh-anugupta sfc-gh-anugupta

Labels
enhancement The issue is a request for improvement or a new feature parity this feature is supported in other drivers status-pr_pending_merge A PR is made and is under review status-triage_done Initial triage done, will be further handled by the driver team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
9 participants
@gdoron @vrrx @sfc-gh-hkapre @sfc-gh-hchaturvedi @sfc-gh-jfan @sfc-gh-dszmolka @sfc-gh-jhollan @sfc-gh-anugupta @markjefford