Intent.IsValid special-case openSession as it does not need to be signed

0xsequence · Dec 1, 2023 · df32814 · df32814
1 parent c39e106
commit df32814
Showing 1 changed file with 12 additions and 6 deletions.
diff --git a/intents/intent.go b/intents/intent.go
@@ -79,6 +79,18 @@ func (intent *Intent) Signers() []string {
 }
 
 func (intent *Intent) IsValid() bool {
+	// Check if the packet is valid
+	var packet packets.BasePacket
+	err := json.Unmarshal(intent.Packet, &packet)
+	if err != nil {
+		return false
+	}
+
+	// OpenSession packets do not require signatures
+	if packet.Code == packets.OpenSessionPacketCode {
+		return packet.IsValid()
+	}
+
 	// Check if there are any signatures
 	if len(intent.signatures) == 0 {
 		return false
@@ -92,12 +104,6 @@ func (intent *Intent) IsValid() bool {
 	}
 
 	// Check if the packet is valid
-	var packet packets.BasePacket
-	err := json.Unmarshal(intent.Packet, &packet)
-	if err != nil {
-		return false
-	}
-
 	return packet.IsValid()
 }