Mutable Metadata via new UpdateMetadata intent kind #1597
Annotations
18 warnings
redundant closure:
xmtp_mls/src/groups/subscriptions.rs#L89
warning: redundant closure
--> xmtp_mls/src/groups/subscriptions.rs:89:13
|
89 | move |message| callback(message),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `callback`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
= note: `#[warn(clippy::redundant_closure)]` on by default
|
useless conversion to the same type: `xmtp_proto::xmtp::mls::database::AccountAddresses`:
xmtp_mls/src/groups/intents.rs#L241
warning: useless conversion to the same type: `xmtp_proto::xmtp::mls::database::AccountAddresses`
--> xmtp_mls/src/groups/intents.rs:241:52
|
241 | allow_list_account_addresses: Some(self.allow_list_account_addresses.clone().into()),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing `.into()`: `self.allow_list_account_addresses.clone()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
= note: `#[warn(clippy::useless_conversion)]` on by default
|
associated function `from_bytes` is never used:
xmtp_mls/src/groups/intents.rs#L250
warning: associated function `from_bytes` is never used
--> xmtp_mls/src/groups/intents.rs:250:19
|
230 | impl UpdateMetadataIntentData {
| ----------------------------- associated function in this implementation
...
250 | pub(crate) fn from_bytes(data: &[u8]) -> Result<Self, IntentError> {
| ^^^^^^^^^^
|
= note: `#[warn(dead_code)]` on by default
|
returning the result of a `let` binding from a block:
xmtp_api_grpc/src/auth_token.rs#L59
warning: returning the result of a `let` binding from a block
--> xmtp_api_grpc/src/auth_token.rs:59:9
|
58 | let token_base64 = base64::engine::general_purpose::STANDARD.encode(&token_bytes);
| ---------------------------------------------------------------------------------- unnecessary `let` binding
59 | token_base64
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
= note: `#[warn(clippy::let_and_return)]` on by default
help: return the expression directly
|
58 ~
59 ~ base64::engine::general_purpose::STANDARD.encode(&token_bytes)
|
|
the borrowed expression implements the required traits:
xmtp_proto/src/gen/xmtp.mls.message_contents.serde.rs#L2629
warning: the borrowed expression implements the required traits
--> xmtp_proto/src/gen/xmtp.mls.message_contents.serde.rs:2629:91
|
2629 | struct_ser.serialize_field("content", pbjson::private::base64::encode(&v).as_str())?;
| ^^ help: change this to: `v`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
xmtp_proto/src/gen/xmtp.mls.message_contents.serde.rs#L483
warning: the borrowed expression implements the required traits
--> xmtp_proto/src/gen/xmtp.mls.message_contents.serde.rs:483:115
|
483 | struct_ser.serialize_field("unsignedLegacyCreateIdentityKey", pbjson::private::base64::encode(&v).as_str())?;
| ^^ help: change this to: `v`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
xmtp_proto/src/gen/xmtp.mls.message_contents.serde.rs#L479
warning: the borrowed expression implements the required traits
--> xmtp_proto/src/gen/xmtp.mls.message_contents.serde.rs:479:99
|
479 | struct_ser.serialize_field("installationKey", pbjson::private::base64::encode(&v).as_str())?;
| ^^ help: change this to: `v`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
xmtp_proto/src/gen/xmtp.message_contents.serde.rs#L2951
warning: the borrowed expression implements the required traits
--> xmtp_proto/src/gen/xmtp.message_contents.serde.rs:2951:86
|
2951 | struct_ser.serialize_field("senderHmac", pbjson::private::base64::encode(&v).as_str())?;
| ^^ help: change this to: `v`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
|
the borrowed expression implements the required traits:
xmtp_proto/src/gen/xmtp.message_contents.serde.rs#L1334
warning: the borrowed expression implements the required traits
--> xmtp_proto/src/gen/xmtp.message_contents.serde.rs:1334:86
|
1334 | struct_ser.serialize_field("v1", pbjson::private::base64::encode(&v).as_str())?;
| ^^ help: change this to: `v`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
= note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default
|
this function has too many arguments (8/7):
src/mls.rs#L48
warning: this function has too many arguments (8/7)
--> src/mls.rs:48:1
|
48 | / pub async fn create_client(
49 | | logger: Box<dyn FfiLogger>,
50 | | host: String,
51 | | is_secure: bool,
... |
56 | | legacy_signed_private_key_proto: Option<Vec<u8>>,
57 | | ) -> Result<Arc<FfiXmtpClient>, GenericError> {
| |_____________________________________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
= note: `#[warn(clippy::too_many_arguments)]` on by default
|
associated function `from_bytes` is never used:
/home/runner/work/libxmtp/libxmtp/xmtp_mls/src/groups/intents.rs#L250
warning: associated function `from_bytes` is never used
--> /home/runner/work/libxmtp/libxmtp/xmtp_mls/src/groups/intents.rs:250:19
|
230 | impl UpdateMetadataIntentData {
| ----------------------------- associated function in this implementation
...
250 | pub(crate) fn from_bytes(data: &[u8]) -> Result<Self, IntentError> {
| ^^^^^^^^^^
|
= note: `#[warn(dead_code)]` on by default
|
Test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions-rs/cargo@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/cargo@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Test:
xmtp_mls/src/groups/intents.rs#L250
associated function `from_bytes` is never used
|
Test
`xmtp_mls` (lib) generated 1 warning
|
Test
`xmtp_mls` (lib test) generated 1 warning (1 duplicate)
|
Test:
xmtp_mls/src/groups/intents.rs#L250
associated function `from_bytes` is never used
|
Test
`xmtp_mls` (lib) generated 1 warning
|