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

Fix the msextensions build #29

Merged
merged 3 commits into from
Apr 10, 2024

Conversation

dmulder
Copy link
Collaborator

@dmulder dmulder commented Apr 10, 2024

get_legacy_kid wasn't implemented for ms_oapxbc.

Implements # .

  • cargo fmt has been run
  • cargo test has been run and passes
  • documentation has been updated with relevant examples (if relevant)

get_legacy_kid wasn't implemented for ms_oapxbc.

Signed-off-by: David Mulder <[email protected]>
@dmulder dmulder force-pushed the dmulder/msextensions_build_fix branch from 1483340 to c91475d Compare April 10, 2024 14:58
@dmulder
Copy link
Collaborator Author

dmulder commented Apr 10, 2024

I've appended a test to the github workflow to prevent this from happening again.

When sending a Windows Hello request, it's
necessary to set a very specific kid value,
otherwise the request is rejected.

Signed-off-by: David Mulder <[email protected]>
@yaleman yaleman merged commit 180962b into kanidm:main Apr 10, 2024
1 check passed
@dmulder dmulder deleted the dmulder/msextensions_build_fix branch April 10, 2024 21:42
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.

2 participants