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

NAS-132659 / 25.04 / Don't alow to enable ssh password login for user where 2fa secret is not available #15157

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sonicaj
Copy link
Member

@sonicaj sonicaj commented Dec 9, 2024

This PR adds changes to not allow to enable ssh password login for user where 2fa secret has not been configured for the user and fixes a couple of bugs on the user pydantic model.

…not available

This commit adds changes to not allow to enable ssh password login for user where 2fa secret has not been configured for the user.
@sonicaj sonicaj requested a review from a team December 9, 2024 10:59
@sonicaj sonicaj self-assigned this Dec 9, 2024
@bugclerk bugclerk changed the title Don't alow to enable ssh password login for user where 2fa secret is not available NAS-132659 / 25.04 / Don't alow to enable ssh password login for user where 2fa secret is not available Dec 9, 2024
@bugclerk
Copy link
Contributor

bugclerk commented Dec 9, 2024

@themylogin
Copy link
Contributor

Can we have an integration test for this?

@yocalebo yocalebo requested a review from anodos325 December 9, 2024 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants