crypto: feature-gate dependency on matrix_qrcode

matrix_sdk/Cargo.toml

@@ -15,9 +15,10 @@ features = ["docs"]
 rustdoc-args = ["--cfg", "feature=\"docs\""]
 
 [features]
-default = ["encryption", "sled_cryptostore", "sled_state_store", "require_auth_for_profile_requests", "native-tls"]
+default = ["encryption", "qrcode", "sled_cryptostore", "sled_state_store", "require_auth_for_profile_requests", "native-tls"]
 
 encryption = ["matrix-sdk-base/encryption"]
+qrcode = ["matrix-sdk-base/qrcode"]
 sled_state_store = ["matrix-sdk-base/sled_state_store"]
 sled_cryptostore = ["matrix-sdk-base/sled_cryptostore"]
 markdown = ["matrix-sdk-base/markdown"]

matrix_sdk/src/client.rs

@@ -130,11 +130,13 @@ use ruma::{
     DeviceIdBox, RoomId, RoomIdOrAliasId, ServerName, UInt, UserId,
 };
 
+#[cfg(all(feature = "qrcode", feature = "encryption"))]
+use crate::verification::QrVerification;
 #[cfg(feature = "encryption")]
 use crate::{
     device::{Device, UserDevices},
     error::RoomKeyImportError,
-    verification::{QrVerification, SasVerification, Verification, VerificationRequest},
+    verification::{SasVerification, Verification, VerificationRequest},
 };
 use crate::{
     error::HttpError,
@@ -2241,6 +2243,7 @@ impl Client {
             matrix_sdk_base::crypto::Verification::SasV1(s) => {
                 SasVerification { inner: s, client: self.clone() }.into()
             }
+            #[cfg(feature = "qrcode")]
             matrix_sdk_base::crypto::Verification::QrV1(qr) => {
                 QrVerification { inner: qr, client: self.clone() }.into()
             }

matrix_sdk/src/lib.rs

@@ -75,6 +75,9 @@ compile_error!("only one of 'native-tls' or 'rustls-tls' features can be enabled
 #[cfg(all(feature = "sso_login", target_arch = "wasm32"))]
 compile_error!("'sso_login' cannot be enabled on 'wasm32' arch");
 
+#[cfg(all(not(feature = "encryption"), feature = "qrcode"))]
+compile_error!("'qrcode' cannot be enabled without 'encryption'");
+
 pub use bytes;
 #[cfg(feature = "encryption")]
 #[cfg_attr(feature = "docs", doc(cfg(encryption)))]

matrix_sdk/src/verification/mod.rs

@@ -29,11 +29,13 @@
 //! string.
 //! * [QrVerification] - Interactive verification using QR codes.
 
+#[cfg(feature = "qrcode")]
 mod qrcode;
 mod requests;
 mod sas;
 
 pub use matrix_sdk_base::crypto::{AcceptSettings, CancelInfo};
+#[cfg(feature = "qrcode")]
 pub use qrcode::QrVerification;
 pub use requests::VerificationRequest;
 pub use sas::SasVerification;
@@ -43,6 +45,7 @@ pub use sas::SasVerification;
 pub enum Verification {
     /// The `m.sas.v1` verification variant.
     SasV1(SasVerification),
+    #[cfg(feature = "qrcode")]
     /// The `m.qr_code.*.v1` verification variant.
     QrV1(QrVerification),
 }
@@ -50,6 +53,7 @@ pub enum Verification {
 impl Verification {
     /// Try to deconstruct this verification enum into a SAS verification.
     pub fn sas(self) -> Option<SasVerification> {
+        #[allow(irrefutable_let_patterns)]
         if let Verification::SasV1(sas) = self {
             Some(sas)
         } else {
@@ -57,6 +61,7 @@ impl Verification {
         }
     }
 
+    #[cfg(feature = "qrcode")]
     /// Try to deconstruct this verification enum into a QR code verification.
     pub fn qr(self) -> Option<QrVerification> {
         if let Verification::QrV1(qr) = self {
@@ -70,6 +75,7 @@ impl Verification {
     pub fn is_done(&self) -> bool {
         match self {
             Verification::SasV1(s) => s.is_done(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(qr) => qr.is_done(),
         }
     }
@@ -78,6 +84,7 @@ impl Verification {
     pub fn is_cancelled(&self) -> bool {
         match self {
             Verification::SasV1(s) => s.is_cancelled(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(qr) => qr.is_cancelled(),
         }
     }
@@ -87,6 +94,7 @@ impl Verification {
     pub fn cancel_info(&self) -> Option<CancelInfo> {
         match self {
             Verification::SasV1(s) => s.cancel_info(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(q) => q.cancel_info(),
         }
     }
@@ -95,6 +103,7 @@ impl Verification {
     pub fn own_user_id(&self) -> &ruma::UserId {
         match self {
             Verification::SasV1(v) => v.own_user_id(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(v) => v.own_user_id(),
         }
     }
@@ -104,6 +113,7 @@ impl Verification {
     pub fn other_user_id(&self) -> &ruma::UserId {
         match self {
             Verification::SasV1(v) => v.inner.other_user_id(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(v) => v.inner.other_user_id(),
         }
     }
@@ -112,6 +122,7 @@ impl Verification {
     pub fn is_self_verification(&self) -> bool {
         match self {
             Verification::SasV1(v) => v.is_self_verification(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(v) => v.is_self_verification(),
         }
     }
@@ -120,6 +131,7 @@ impl Verification {
     pub fn we_started(&self) -> bool {
         match self {
             Verification::SasV1(s) => s.we_started(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(q) => q.we_started(),
         }
     }
@@ -131,6 +143,7 @@ impl From<SasVerification> for Verification {
     }
 }
 
+#[cfg(feature = "qrcode")]
 impl From<QrVerification> for Verification {
     fn from(qr: QrVerification) -> Self {
         Self::QrV1(qr)

matrix_sdk/src/verification/requests.rs

@@ -15,7 +15,9 @@
 use matrix_sdk_base::crypto::{CancelInfo, VerificationRequest as BaseVerificationRequest};
 use ruma::events::key::verification::VerificationMethod;
 
-use super::{QrVerification, SasVerification};
+#[cfg(feature = "qrcode")]
+use super::QrVerification;
+use super::SasVerification;
 use crate::{Client, Result};
 
 /// An object controlling the interactive verification flow.
@@ -112,6 +114,7 @@ impl VerificationRequest {
         Ok(())
     }
 
+    #[cfg(feature = "qrcode")]
     /// Generate a QR code
     pub async fn generate_qr_code(&self) -> Result<Option<QrVerification>> {
         Ok(self

matrix_sdk_base/Cargo.toml

@@ -17,6 +17,7 @@ rustdoc-args = ["--cfg", "feature=\"docs\""]
 [features]
 default = []
 encryption = ["matrix-sdk-crypto"]
+qrcode = ["matrix-sdk-crypto/qrcode"]
 sled_state_store = ["sled", "pbkdf2", "hmac", "sha2", "rand", "chacha20poly1305"]
 sled_cryptostore = ["matrix-sdk-crypto/sled_cryptostore"]
 markdown = ["ruma/markdown"]

matrix_sdk_crypto/Cargo.toml

@@ -16,11 +16,12 @@ rustdoc-args = ["--cfg", "feature=\"docs\""]
 
 [features]
 default = []
+qrcode = ["matrix-qrcode"]
 sled_cryptostore = ["sled"]
 docs = ["sled_cryptostore"]
 
 [dependencies]
-matrix-qrcode = { version = "0.1.0", path = "../matrix_qrcode" }
+matrix-qrcode = { version = "0.1.0", path = "../matrix_qrcode" , optional = true}
 matrix-sdk-common = { version = "0.3.0", path = "../matrix_sdk_common" }
 ruma = { version = "0.3.0", features = ["client-api-c", "unstable-pre-spec"] }
 

matrix_sdk_crypto/src/lib.rs

@@ -49,6 +49,7 @@ pub use identities::{
     ReadOnlyUserIdentities, ReadOnlyUserIdentity, UserDevices, UserIdentities, UserIdentity,
 };
 pub use machine::OlmMachine;
+#[cfg(feature = "qrcode")]
 pub use matrix_qrcode;
 pub(crate) use olm::ReadOnlyAccount;
 pub use olm::{CrossSigningStatus, EncryptionSettings};
@@ -57,6 +58,6 @@ pub use requests::{
     OutgoingVerificationRequest, RoomMessageRequest, ToDeviceRequest, UploadSigningKeysRequest,
 };
 pub use store::{CrossSigningKeyExport, CryptoStoreError, SecretImportError};
-pub use verification::{
+#[cfg(feature = "qrcode")]
-    AcceptSettings, CancelInfo, QrVerification, Sas, Verification, VerificationRequest,
+pub use verification::QrVerification;
-};
+pub use verification::{AcceptSettings, CancelInfo, Sas, Verification, VerificationRequest};

matrix_sdk_crypto/src/verification/cache.rs

@@ -20,10 +20,9 @@ use ruma::{DeviceId, UserId};
 use tracing::trace;
 
 use super::{event_enums::OutgoingContent, Sas, Verification};
-use crate::{
+#[cfg(feature = "qrcode")]
-    OutgoingRequest, OutgoingVerificationRequest, QrVerification, RoomMessageRequest,
+use crate::QrVerification;
-    ToDeviceRequest,
+use crate::{OutgoingRequest, OutgoingVerificationRequest, RoomMessageRequest, ToDeviceRequest};
-};
 
 #[derive(Clone, Debug)]
 pub struct VerificationCache {
@@ -55,10 +54,12 @@ impl VerificationCache {
         self.insert(sas);
     }
 
+    #[cfg(feature = "qrcode")]
     pub fn insert_qr(&self, qr: QrVerification) {
         self.insert(qr)
     }
 
+    #[cfg(feature = "qrcode")]
     pub fn get_qr(&self, sender: &UserId, flow_id: &str) -> Option<QrVerification> {
         self.get(sender, flow_id).and_then(|v| {
             if let Verification::QrV1(qr) = v {
@@ -91,6 +92,7 @@ impl VerificationCache {
                     .value()
                     .iter()
                     .filter_map(|s| {
+                        #[allow(irrefutable_let_patterns)]
                         if let Verification::SasV1(s) = s.value() {
                             s.cancel_if_timed_out()
                         } else {
@@ -106,6 +108,7 @@ impl VerificationCache {
 
     pub fn get_sas(&self, user_id: &UserId, flow_id: &str) -> Option<Sas> {
         self.get(user_id, flow_id).and_then(|v| {
+            #[allow(irrefutable_let_patterns)]
             if let Verification::SasV1(sas) = v {
                 Some(sas)
             } else {

matrix_sdk_crypto/src/verification/machine.rs

@@ -387,6 +387,7 @@ impl VerificationMachine {
                                 // This won't produce an outgoing content
                                 let _ = sas.receive_any_event(event.sender(), &content);
                             }
+                            #[cfg(feature = "qrcode")]
                             Verification::QrV1(qr) => qr.receive_cancel(event.sender(), c),
                         }
                     }
@@ -485,6 +486,7 @@ impl VerificationMachine {
                                 self.mark_sas_as_done(sas, content).await?;
                             }
                         }
+                        #[cfg(feature = "qrcode")]
                         Some(Verification::QrV1(qr)) => {
                             let (cancellation, request) = qr.receive_done(c).await?;
 

matrix_sdk_crypto/src/verification/mod.rs

@@ -15,6 +15,7 @@
 mod cache;
 mod event_enums;
 mod machine;
+#[cfg(feature = "qrcode")]
 mod qrcode;
 mod requests;
 mod sas;
@@ -27,6 +28,7 @@ use std::{
 use event_enums::OutgoingContent;
 pub use machine::VerificationMachine;
 use matrix_sdk_common::locks::Mutex;
+#[cfg(feature = "qrcode")]
 pub use qrcode::QrVerification;
 pub use requests::VerificationRequest;
 use ruma::{
@@ -115,6 +117,7 @@ impl VerificationStore {
 pub enum Verification {
     /// The `m.sas.v1` verification variant.
     SasV1(Sas),
+    #[cfg(feature = "qrcode")]
     /// The `m.qr_code.*.v1` verification variant.
     QrV1(QrVerification),
 }
@@ -122,6 +125,7 @@ pub enum Verification {
 impl Verification {
     /// Try to deconstruct this verification enum into a SAS verification.
     pub fn sas_v1(self) -> Option<Sas> {
+        #[allow(irrefutable_let_patterns)]
         if let Verification::SasV1(sas) = self {
             Some(sas)
         } else {
@@ -129,6 +133,7 @@ impl Verification {
         }
     }
 
+    #[cfg(feature = "qrcode")]
     /// Try to deconstruct this verification enum into a QR code verification.
     pub fn qr_v1(self) -> Option<QrVerification> {
         if let Verification::QrV1(qr) = self {
@@ -142,6 +147,7 @@ impl Verification {
     pub fn is_done(&self) -> bool {
         match self {
             Verification::SasV1(s) => s.is_done(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(qr) => qr.is_done(),
         }
     }
@@ -150,6 +156,7 @@ impl Verification {
     pub fn flow_id(&self) -> &str {
         match self {
             Verification::SasV1(s) => s.flow_id().as_str(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(qr) => qr.flow_id().as_str(),
         }
     }
@@ -158,6 +165,7 @@ impl Verification {
     pub fn is_cancelled(&self) -> bool {
         match self {
             Verification::SasV1(s) => s.is_cancelled(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(qr) => qr.is_cancelled(),
         }
     }
@@ -166,6 +174,7 @@ impl Verification {
     pub fn user_id(&self) -> &UserId {
         match self {
             Verification::SasV1(v) => v.user_id(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(v) => v.user_id(),
         }
     }
@@ -174,6 +183,7 @@ impl Verification {
     pub fn other_user(&self) -> &UserId {
         match self {
             Verification::SasV1(s) => s.other_user_id(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(qr) => qr.other_user_id(),
         }
     }
@@ -182,6 +192,7 @@ impl Verification {
     pub fn is_self_verification(&self) -> bool {
         match self {
             Verification::SasV1(v) => v.is_self_verification(),
+            #[cfg(feature = "qrcode")]
             Verification::QrV1(v) => v.is_self_verification(),
         }
     }
@@ -193,6 +204,7 @@ impl From<Sas> for Verification {
     }
 }
 
+#[cfg(feature = "qrcode")]
 impl From<QrVerification> for Verification {
     fn from(qr: QrVerification) -> Self {
         Self::QrV1(qr)
@@ -376,6 +388,7 @@ pub struct IdentitiesBeingVerified {
 }
 
 impl IdentitiesBeingVerified {
+    #[cfg(feature = "qrcode")]
     async fn can_sign_devices(&self) -> bool {
         self.private_identity.can_sign_devices().await
     }

matrix_sdk_crypto/src/verification/requests.rs

@@ -17,8 +17,12 @@ use std::{
     time::Duration,
 };
 
+#[cfg(feature = "qrcode")]
 use matrix_qrcode::QrVerificationData;
 use matrix_sdk_common::{instant::Instant, uuid::Uuid};
+#[cfg(feature = "qrcode")]
+use ruma::DeviceKeyAlgorithm;
+
 use ruma::{
     events::{
         key::verification::{
@@ -32,7 +36,7 @@ use ruma::{
         AnyMessageEventContent, AnyToDeviceEventContent,
     },
     to_device::DeviceIdOrAllDevices,
-    DeviceId, DeviceIdBox, DeviceKeyAlgorithm, MilliSecondsSinceUnixEpoch, RoomId, UserId,
+    DeviceId, DeviceIdBox, MilliSecondsSinceUnixEpoch, RoomId, UserId,
 };
 use tracing::{info, trace, warn};
 
@@ -41,8 +45,12 @@ use super::{
     event_enums::{
         CancelContent, DoneContent, OutgoingContent, ReadyContent, RequestContent, StartContent,
     },
+    CancelInfo, Cancelled, FlowId, VerificationStore,
+};
+#[cfg(feature = "qrcode")]
+use super::{
     qrcode::{QrVerification, ScanError},
-    CancelInfo, Cancelled, FlowId, IdentitiesBeingVerified, VerificationStore,
+    IdentitiesBeingVerified,
 };
 use crate::{
     olm::{PrivateCrossSigningIdentity, ReadOnlyAccount},
@@ -299,6 +307,7 @@ impl VerificationRequest {
         matches!(&*self.inner.lock().unwrap(), InnerRequest::Cancelled(_))
     }
 
+    #[cfg(feature = "qrcode")]
     /// Generate a QR code that can be used by another client to start a QR code
     /// based verification.
     pub async fn generate_qr_code(&self) -> Result<Option<QrVerification>, CryptoStoreError> {
@@ -308,7 +317,8 @@ impl VerificationRequest {
             .generate_qr_code(self.we_started, self.inner.clone().into())
             .await
     }
-
+    ///
+    #[cfg(feature = "qrcode")]
     /// Start a QR code verification by providing a scanned QR code for this
     /// verification flow.
     ///
@@ -452,6 +462,7 @@ impl VerificationRequest {
         {
             match verification {
                 crate::Verification::SasV1(s) => s.cancel_with_code(cancel_code),
+                #[cfg(feature = "qrcode")]
                 crate::Verification::QrV1(q) => q.cancel_with_code(cancel_code),
             };
         }
@@ -720,6 +731,7 @@ impl InnerRequest {
         });
     }
 
+    #[cfg(feature = "qrcode")]
     async fn generate_qr_code(
         &self,
         we_started: bool,
@@ -938,6 +950,7 @@ impl RequestState<Ready> {
         )
     }
 
+    #[cfg(feature = "qrcode")]
     async fn generate_qr_code(
         &self,
         we_started: bool,
@@ -1141,6 +1154,7 @@ impl RequestState<Ready> {
                     }
                 }
             }
+            #[cfg(feature = "qrcode")]
             StartMethod::ReciprocateV1(_) => {
                 if let Some(qr_verification) =
                     self.verification_cache.get_qr(sender, content.flow_id())