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

Stable 0.7.x Hello Pin changes via passwd command #321

Merged
merged 4 commits into from
Dec 10, 2024

Conversation

dmulder
Copy link
Collaborator

@dmulder dmulder commented Dec 10, 2024

Fixes #279

Checklist

  • This pr contains no AI generated code
  • cargo fmt has been run
  • cargo clippy has been run
  • A functionality test has been added
  • make test has been run and passes

This is just a remnant of the Kanidm fork.

Signed-off-by: David Mulder <[email protected]>
This implements Pin changes via pam (a user will
call the terminal `passwd` command). This will
require more interaction than a typical call to
`passwd`, since we need to authenticate to Entra
Id to register the new Pin. This extra
authentication step forces the token to have an
ngcmfa amr.

Signed-off-by: David Mulder <[email protected]>
Signed-off-by: David Mulder <[email protected]>
Signed-off-by: David Mulder <[email protected]>
@dmulder dmulder merged commit fe7972e into stable-0.7.x Dec 10, 2024
5 checks passed
@dmulder dmulder deleted the stable-0.7.x_hello_pin_change branch December 10, 2024 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant