RockteMQ-AI commented on issue #281:
URL: https://github.com/apache/rocketmq-mqtt/issues/281#issuecomment-5250119226

   **Issue Evaluation**
   
   Category: `type/bug` | Status: **Confirmed**
   
   The MQTT 5.0 specification defines the Session Present flag in CONNACK to 
indicate whether the server has a stored session for the client. 
`ConnectHandler` always sets `session-present=false`, which violates the 
protocol when a client reconnects with `Clean Start=false` and the server does 
have a stored session.
   
   **Severity:** Medium — protocol compliance issue; affects clients that rely 
on session resumption.
   
   **Suggested Fix:** Check the session store in `ConnectHandler` before 
building the CONNACK, and set the Session Present flag based on whether a 
session exists for the client ID.
   
   ---
   *Automated evaluation by github-manager*


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to