Skip to content

Update webauthn-rs 6.0 to use crypto-glue 2.0 - #583

Open
Firstyear wants to merge 5 commits into
kanidm:6.0-dev-drop-opensslfrom
Firstyear:20260825-update-glue
Open

Firstyear wants to merge 5 commits into
kanidm:6.0-dev-drop-opensslfrom
Firstyear:20260825-update-glue

Conversation

@Firstyear

Copy link
Copy Markdown
Member

Crypto-glue 2.0 has updated to the latest set of rustcrypto crate releases. This updates webauthn-rs to use that new version.

Checklist

  • This PR contains no AI generated code
  • cargo test has been run and passes
  • documentation has been updated with relevant examples (if relevant)

Crypto-glue 2.0 has updated to the latest set of rustcrypto crate releases.
This updates webauthn-rs to use that new version.
@infiniwave infiniwave mentioned this pull request Aug 26, 2026
3 tasks
@Firstyear

Copy link
Copy Markdown
Member Author

@micolous Want to have a look over this?

@yaleman yaleman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just nits otherwise woo 😄

block_padding::generic_array::{
sequence::Split,
block_padding::array::{
// sequence::Split,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// sequence::Split,

block_padding::generic_array::{
sequence::Split,
block_padding::array::{
// sequence::Split,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// sequence::Split,

Comment on lines +209 to +215
/*
impl From<crypto_glue::rand::Error> for WebauthnCError {
fn from(value: crypto_glue::rand::Error) -> Self {
Self::Rand(value.to_string())
}
}
*/

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/*
impl From<crypto_glue::rand::Error> for WebauthnCError {
fn from(value: crypto_glue::rand::Error) -> Self {
Self::Rand(value.to_string())
}
}
*/

serial_number,
validity,
root_subject.clone(),
// root_subject.clone(),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// root_subject.clone(),

// root_subject.clone(),
pub_key.clone(),
ca_key,
// ca_key,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// ca_key,

Comment thread Cargo.toml
exclude = ["compat_tester/webauthn-rs-demo-wasm", "tutorial/wasm"]

# [patch.crates-io]
[patch.crates-io]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[patch.crates-io]
# [patch.crates-io]

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