[JAMES-3624] RFC 8887 (JMAP over WebSocket) Request needs property 'id' (is 'requestId')

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.7.0
• Assignee: Unassigned
• Reporter: Daniel Gultsch
• Created: 04/Aug/21 08:35
• Updated: 06/Aug/21 03:19
• Resolved: 06/Aug/21 03:19

Description

The WebSocket message 'Request' has a property called 'id' according to the spec. (Responses will mirror that property in a property called 'requestId'.) James however calls this 'requestId'

 

The unit test here:

https://github.com/apache/james-project/blob/master/server/protocols/jmap-rfc-8621-integration-tests/jmap-rfc-8621-integration-tests-common/src/main/scala/org/apache/james/jmap/rfc8621/contract/WebSocketContract.scala#L77

 

needs to read:

"id": "req-36",

 

(I messed this up in my JMAP library as well. The fix is here: https://github.com/iNPUTmice/jmap/commit/58fa163b8d55e3a6e5be4cf3bbfaca4700963b93 )

Issue Links

• links to: GitHub Pull Request #573