temporary comment for testing

xmtp · Apr 4, 2024 · 93dc30f · 93dc30f
1 parent fa33f5e
commit 93dc30f
Show file tree

Hide file tree

Showing 3 changed files with 45 additions and 42 deletions.
diff --git a/openmls/src/group/errors.rs b/openmls/src/group/errors.rs
@@ -374,6 +374,9 @@ pub enum ProposalValidationError {
     /// See [`PskError`] for more details.
     #[error(transparent)]
     Psk(#[from] PskError),
+    /// Leaf node
+    #[error("something with leaf node")]
+    LeafNodeValidationError,
 }
 
 /// External Commit validaton error

diff --git a/openmls/src/group/public_group/validation.rs b/openmls/src/group/public_group/validation.rs
@@ -315,32 +315,32 @@ impl PublicGroup {
                     .map_err(|_| ProposalValidationError::InsufficientCapabilities)?;
             }
 
-            // Check that all extensions are contained in the capabilities.
-            if !capabilities.contain_extensions(leaf_node.extensions()) {
-                return Err(ProposalValidationError::InsufficientCapabilities);
-            }
-
-            // Check that the capabilities contain the leaf node's credential type.
-            if !capabilities.contains_credential(&leaf_node.credential().credential_type()) {
-                return Err(ProposalValidationError::InsufficientCapabilities);
-            }
-
-            // Check that the credential type is supported by all members of the group.
-            if !group_leaf_nodes.all(|node| {
-                node.capabilities()
-                    .contains_credential(&leaf_node.credential().credential_type())
-            }) {
-                return Err(ProposalValidationError::InsufficientCapabilities);
-            }
-
-            // Check that the capabilities field of this LeafNode indicates
-            // support for all the credential types currently in use by other
-            // members.
-            if !group_leaf_nodes
-                .all(|node| capabilities.contains_credential(&node.credential().credential_type()))
-            {
-                return Err(ProposalValidationError::InsufficientCapabilities);
-            }
+            // // Check that all extensions are contained in the capabilities.
+            // if !capabilities.contain_extensions(leaf_node.extensions()) {
+            //     return Err(ProposalValidationError::InsufficientCapabilities);
+            // }
+
+            // // Check that the capabilities contain the leaf node's credential type.
+            // if !capabilities.contains_credential(&leaf_node.credential().credential_type()) {
+            //     return Err(ProposalValidationError::InsufficientCapabilities);
+            // }
+
+            // // Check that the credential type is supported by all members of the group.
+            // if !group_leaf_nodes.all(|node| {
+            //     node.capabilities()
+            //         .contains_credential(&leaf_node.credential().credential_type())
+            // }) {
+            //     return Err(ProposalValidationError::InsufficientCapabilities);
+            // }
+
+            // // Check that the capabilities field of this LeafNode indicates
+            // // support for all the credential types currently in use by other
+            // // members.
+            // if !group_leaf_nodes
+            //     .all(|node| capabilities.contains_credential(&node.credential().credential_type()))
+            // {
+            //     return Err(ProposalValidationError::InsufficientCapabilities);
+            // }
         }
         Ok(())
     }

diff --git a/openmls/src/treesync/node/leaf_node/capabilities.rs b/openmls/src/treesync/node/leaf_node/capabilities.rs
@@ -141,22 +141,22 @@ impl Capabilities {
         {
             return Err(LeafNodeValidationError::UnsupportedExtensions);
         }
-        // Check if all required proposals are supported.
-        if required_capabilities
-            .proposal_types()
-            .iter()
-            .any(|p| !(self.proposals().contains(p) || default_proposals().contains(p)))
-        {
-            return Err(LeafNodeValidationError::UnsupportedProposals);
-        }
-        // Check if all required credential types are supported.
-        if required_capabilities
-            .credential_types()
-            .iter()
-            .any(|c| !(self.credentials().contains(c) || default_credentials().contains(c)))
-        {
-            return Err(LeafNodeValidationError::UnsupportedCredentials);
-        }
+        // // Check if all required proposals are supported.
+        // if required_capabilities
+        //     .proposal_types()
+        //     .iter()
+        //     .any(|p| !(self.proposals().contains(p) || default_proposals().contains(p)))
+        // {
+        //     return Err(LeafNodeValidationError::UnsupportedProposals);
+        // }
+        // // Check if all required credential types are supported.
+        // if required_capabilities
+        //     .credential_types()
+        //     .iter()
+        //     .any(|c| !(self.credentials().contains(c) || default_credentials().contains(c)))
+        // {
+        //     return Err(LeafNodeValidationError::UnsupportedCredentials);
+        // }
         Ok(())
     }