Skip to content

Add xmtp_xid

Add xmtp_xid #795

Triggered via pull request March 25, 2024 17:13
Status Failure
Total duration 12s
Artifacts

ci-image.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

1 error and 9 warnings
build
Process completed with exit code 1.
associated function `create_to_be_signed` is never used: xmtp_xid/src/lib.rs#L24
warning: associated function `create_to_be_signed` is never used --> xmtp_xid/src/lib.rs:24:19 | 21 | impl Identity { | ------------- associated function in this implementation ... 24 | pub(crate) fn create_to_be_signed(account_address: String) -> Result<Self, IdentityError> { | ^^^^^^^^^^^^^^^^^^^
fields `account_address` and `installation_keys` are never read: xmtp_xid/src/lib.rs#L15
warning: fields `account_address` and `installation_keys` are never read --> xmtp_xid/src/lib.rs:15:16 | 14 | pub struct Identity { | -------- fields in this struct 15 | pub(crate) account_address: Address, | ^^^^^^^^^^^^^^^ 16 | pub(crate) installation_keys: SignatureKeyPair, | ^^^^^^^^^^^^^^^^^ | = note: `#[warn(dead_code)]` on by default
redundant closure: xmtp_mls/src/groups/subscriptions.rs#L77
warning: redundant closure --> xmtp_mls/src/groups/subscriptions.rs:77:13 | 77 | 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
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#L391
warning: the borrowed expression implements the required traits --> xmtp_proto/src/gen/xmtp.mls.message_contents.serde.rs:391:115 | 391 | 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#L387
warning: the borrowed expression implements the required traits --> xmtp_proto/src/gen/xmtp.mls.message_contents.serde.rs:387:99 | 387 | 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#L2934
warning: the borrowed expression implements the required traits --> xmtp_proto/src/gen/xmtp.message_contents.serde.rs:2934:86 | 2934 | 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