OCP.1: allow for OCP.1 protocol versions greater than 1

lhoward · lhoward · commit e3981babfb74 · 2026-02-18T11:45:31.000+11:00
AES70-3-2024 says that the OCP.1 protocol version should always be 1, but some
clients send higher versions (presumably to match the AES70 revision). accept
those; we will need to use different heuristics if the actual wire protocol
changes.
diff --git a/Sources/SwiftOCA/OCF/Messages/Header.swift b/Sources/SwiftOCA/OCF/Messages/Header.swift
@@ -50,7 +50,11 @@ public struct Ocp1Header: Codable, Sendable, _Ocp1Codable {
       OcaUint16(bigEndian: $0.loadUnaligned(fromByteOffset: 0, as: OcaUint16.self))
     }
 
-    guard protocolVersion == Ocp1ProtocolVersion else {
+    // AES70-3-2024 says that the OCP.1 protocol version should always be 1,
+    // but some clients send higher versions (presumably to match the AES70
+    // revision). accept those; we will need to use different heuristics if the
+    // actual wire protocol changes.
+    guard protocolVersion >= Ocp1ProtocolVersion else {
       throw Ocp1Error.invalidProtocolVersion
     }
 

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,11 @@ public struct Ocp1Header: Codable, Sendable, _Ocp1Codable {`
`50`	`50`	`OcaUint16(bigEndian: $0.loadUnaligned(fromByteOffset: 0, as: OcaUint16.self))`
`51`	`51`	`}`
`52`	`52`
`53`		`- guard protocolVersion == Ocp1ProtocolVersion else {`
	`53`	`+ // AES70-3-2024 says that the OCP.1 protocol version should always be 1,`
	`54`	`+ // but some clients send higher versions (presumably to match the AES70`
	`55`	`+ // revision). accept those; we will need to use different heuristics if the`
	`56`	`+ // actual wire protocol changes.`
	`57`	`+ guard protocolVersion >= Ocp1ProtocolVersion else {`
`54`	`58`	`throw Ocp1Error.invalidProtocolVersion`
`55`	`59`	`}`
`56`	`60`