Support BBS+ and JWP #4252
Annotations
10 errors and 2 warnings
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L109
error: redundant field names in struct initialization
--> identity_jose/src/jwk/jwk_ext.rs:109:13
|
109 | kty: kty,
| ^^^^^^^^ help: replace it with: `kty`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
= note: `-D clippy::redundant-field-names` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::redundant_field_names)]`
|
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L116
error: redundant field names in struct initialization
--> identity_jose/src/jwk/jwk_ext.rs:116:13
|
116 | x5u: x5u,
| ^^^^^^^^ help: replace it with: `x5u`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L120
error: redundant field names in struct initialization
--> identity_jose/src/jwk/jwk_ext.rs:120:13
|
120 | params: params
| ^^^^^^^^^^^^^^ help: replace it with: `params`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L150
error: redundant field names in struct initialization
--> identity_jose/src/jwk/jwk_ext.rs:150:13
|
150 | alg: alg,
| ^^^^^^^^ help: replace it with: `alg`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_field_names
|
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L24
error: an implementation of `From` is preferred since it gives you `Into<_>` for free where the reverse isn't true
--> identity_jose/src/jwk/jwk_ext.rs:24:1
|
24 | impl Into<KeyOps> for JwkOperation {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: `impl From<Local> for Foreign` is allowed by the orphan rules, for more information see
https://doc.rust-lang.org/reference/items/implementations.html#trait-implementation-coherence
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#from_over_into
= note: `-D clippy::from-over-into` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::from_over_into)]`
help: replace the `Into` implementation with `From<jwk::key_operation::JwkOperation>`
|
24 ~ impl From<JwkOperation> for KeyOps {
25 ~ fn from(val: JwkOperation) -> Self {
26 ~ match val {
27 ~ JwkOperation::Sign => KeyOps::Sign,
28 ~ JwkOperation::Verify => KeyOps::Verify,
29 ~ JwkOperation::Encrypt => KeyOps::Encrypt,
30 ~ JwkOperation::Decrypt => KeyOps::Decrypt,
31 ~ JwkOperation::WrapKey => KeyOps::WrapKey,
32 ~ JwkOperation::UnwrapKey => KeyOps::UnwrapKey,
33 ~ JwkOperation::DeriveKey => KeyOps::DeriveKey,
34 ~ JwkOperation::DeriveBits => KeyOps::DeriveBits,
35 ~ JwkOperation::ProofGeneration => KeyOps::ProofGeneration,
36 ~ JwkOperation::ProofVerification => KeyOps::ProofVerification,
|
|
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L52
error: an implementation of `From` is preferred since it gives you `Into<_>` for free where the reverse isn't true
--> identity_jose/src/jwk/jwk_ext.rs:52:1
|
52 | impl Into<PKUse> for JwkUse {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: `impl From<Local> for Foreign` is allowed by the orphan rules, for more information see
https://doc.rust-lang.org/reference/items/implementations.html#trait-implementation-coherence
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#from_over_into
help: replace the `Into` implementation with `From<jwk::key_use::JwkUse>`
|
52 ~ impl From<JwkUse> for PKUse {
53 ~ fn from(val: JwkUse) -> Self {
54 ~ match val {
55 ~ JwkUse::Signature => PKUse::Signature,
56 ~ JwkUse::Encryption => PKUse::Encryption,
57 ~ JwkUse::Proof => PKUse::Proof,
|
|
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L80
error: unneeded `return` statement
--> identity_jose/src/jwk/jwk_ext.rs:80:17
|
80 | return Self::Error::KeyError("Invalid crv!") })?,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
= note: `-D clippy::needless-return` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::needless_return)]`
help: remove `return`
|
80 | Self::Error::KeyError("Invalid crv!") })?,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
|
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L97
error: unneeded `return` statement
--> identity_jose/src/jwk/jwk_ext.rs:97:25
|
97 | return Self::Error::InvalidClaim("x5u");
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
help: remove `return`
|
97 - return Self::Error::InvalidClaim("x5u");
97 + Self::Error::InvalidClaim("x5u")
|
|
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L110
error: using `Option.and_then(|x| Some(y))`, which is more succinctly expressed as `map(|x| y)`
--> identity_jose/src/jwk/jwk_ext.rs:110:19
|
110 | use_: value.pk_use.and_then(|u| Some(JwkUse::from(u))),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `value.pk_use.map(|u| JwkUse::from(u))`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bind_instead_of_map
= note: `-D clippy::bind-instead-of-map` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(clippy::bind_instead_of_map)]`
|
core clippy check:
identity_jose/src/jwk/jwk_ext.rs#L111
error: using `Option.and_then(|x| Some(y))`, which is more succinctly expressed as `map(|x| y)`
--> identity_jose/src/jwk/jwk_ext.rs:111:22
|
111 | key_ops: value.key_ops.and_then(|vec_key_ops| {
| ______________________^
112 | | Some(vec_key_ops.into_iter().map(JwkOperation::from).collect())
113 | | }),
| |______________^ help: try: `value.key_ops.map(|vec_key_ops| vec_key_ops.into_iter().map(JwkOperation::from).collect())`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bind_instead_of_map
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, jetli/wasm-bindgen-action@24ba6f9fff570246106ac3f80f35185600c3f6c9, actions-rs-plus/clippy-check@b09a9c37c9df7db8b1a5d52e8fe8e0b6e3d574c4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
The following actions uses node12 which is deprecated and will be forced to run on node16: jetli/wasm-bindgen-action@24ba6f9fff570246106ac3f80f35185600c3f6c9. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
The logs for this run have expired and are no longer available.
Loading