You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This seems to violate the Stomp specification which says
The transaction header is REQUIRED, and the transaction identifier will be used for SEND, COMMIT, ABORT, ACK, and NACK frames to bind them to the named transaction.
and
The transaction header is REQUIRED and MUST specify the identifier of the transaction to (commit|abort).
In
Stomp\Protocol\Protocol
the methods for transactionBEGIN
,COMMIT
&ABORT
commands all allownull
for the transaction ID.https://github.com/stomp-php/stomp-php/blob/master/src/Protocol/Protocol.php#L161
https://github.com/stomp-php/stomp-php/blob/master/src/Protocol/Protocol.php#L174
https://github.com/stomp-php/stomp-php/blob/master/src/Protocol/Protocol.php#L187
This seems to violate the Stomp specification which says
and
https://stomp.github.io/stomp-specification-1.2.html#BEGIN
https://stomp.github.io/stomp-specification-1.2.html#COMMIT
https://stomp.github.io/stomp-specification-1.2.html#ABORT
Either the
$transactionId
should be required or the method should throw an Exception if the$transactionId
isnull
.The text was updated successfully, but these errors were encountered: