Skip to content

Comments

[FIXED] MQTT: Did not reject Packet Identifier set to 0 for SUB/UNSUB#7805

Merged
neilalexander merged 1 commit intomainfrom
fix_7803
Feb 9, 2026
Merged

[FIXED] MQTT: Did not reject Packet Identifier set to 0 for SUB/UNSUB#7805
neilalexander merged 1 commit intomainfrom
fix_7803

Conversation

@kozlovic
Copy link
Member

@kozlovic kozlovic commented Feb 7, 2026

According to the specification, the server should reject a SUB and UNSUB protocol with a packet indentifier equal to 0.

Resolves #7803

Signed-off-by: Ivan Kozlovic ivan@synadia.com

According to the specification, the server should reject a SUB
and UNSUB protocol with a packet indentifier equal to 0.

Resolves #7803

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
@kozlovic kozlovic requested a review from a team as a code owner February 7, 2026 21:36
Copy link
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@neilalexander neilalexander merged commit beabe44 into main Feb 9, 2026
107 of 114 checks passed
@neilalexander neilalexander deleted the fix_7803 branch February 9, 2026 10:17
neilalexander added a commit that referenced this pull request Feb 16, 2026
Includes the following:

- #7780
- #7784
- #7782
- #7783
- #7787
- #7789
- #7793
- #7797
- #7798
- #7799
- #7790
- #7805
- #7810
- #7811
- #7812
- #7809
- #7724
- #7815
- #7816
- #7818
- #7819
- #7820
- #7795
- #7825
- #7828
- #7835
- #7837

Signed-off-by: Neil Twigg <neil@nats.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MQTT: SUBSCRIBE/UNSUBSCRIBE accept Packet Identifier = 0 (violates MQTT 3.1.1 §2.3.1)

2 participants