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

SNOW-1367369: Allow passing passcode for DUO MFA #838

Closed
mcastany opened this issue May 6, 2024 · 3 comments
Closed

SNOW-1367369: Allow passing passcode for DUO MFA #838

mcastany opened this issue May 6, 2024 · 3 comments
Assignees
Labels
duplicate This issue or pull request already exists question Issue is a usage/other question rather than a bug status-triage_done Initial triage done, will be further handled by the driver team

Comments

@mcastany
Copy link

mcastany commented May 6, 2024

What is the current behavior?

I cannot configure the SDK with passcode and I'm spamming DUO push notifications to my device everytime I want to execute a script

What is the desired behavior?

I can setup my DUO mfa passcode and be able to connect to Snowflake without a push notification

@github-actions github-actions bot changed the title Allow passing passcode for DUO MFA SNOW-1367369: Allow passing passcode for DUO MFA May 6, 2024
@sfc-gh-dszmolka sfc-gh-dszmolka self-assigned this May 6, 2024
@sfc-gh-dszmolka sfc-gh-dszmolka added question Issue is a usage/other question rather than a bug status-triage_done Initial triage done, will be further handled by the driver team labels May 6, 2024
@sfc-gh-dszmolka
Copy link
Collaborator

hi - I know the current behaviour is mildly annoying .. fortunately, the works are already in progress to add MFA caching capability for this driver too. Tracking issue is #493, I recommend subscribing to those updates.

After the capability is implemented and a new driver version is released on top of the (future) merged PR, you won't need to spam and be spammed by DUO notifications after enabling the MFA caching feature on the server-side and driver side too. (server-side already supports this, however the snowflake-sdk driver not yet)

Hope this answers your question - also would like to recommend closing this issue and following the other one linked here.

@sfc-gh-dszmolka sfc-gh-dszmolka added the duplicate This issue or pull request already exists label May 6, 2024
@mcastany
Copy link
Author

mcastany commented May 6, 2024

@sfc-gh-dszmolka TBH, I don't mind the spam that much, but the other day my account got locked. Do you know if there's any limit of push notifications from Duo?

@sfc-gh-dszmolka
Copy link
Collaborator

this is two independent things. For one, your Snowflake user will get locked temporarily, regardless of the authentication method and MFA/no MFA, if there are too many unsuccessful login attempts. This temporal lock should also disappear on its own after 15 minutes.

For the rate limit on the DUO authentication API, that's not managed by Snowflake but they probably have a rate limit. It should be transparent to the Snowflake user.

Hope this clarifies but please let me know if you have any other questions. Otherwise, I suggest closing this duplicate issue and following the linked one for the feature implementation.

@mcastany mcastany closed this as completed May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists question Issue is a usage/other question rather than a bug status-triage_done Initial triage done, will be further handled by the driver team
Projects
None yet
Development

No branches or pull requests

2 participants