Skip to content

ARTEMIS-5540 clarify doc for MQTT default session expiry interval#5785

Merged
jbertram merged 1 commit intoapache:mainfrom
john9x:ARTEMIS-5540
Jun 20, 2025
Merged

ARTEMIS-5540 clarify doc for MQTT default session expiry interval#5785
jbertram merged 1 commit intoapache:mainfrom
john9x:ARTEMIS-5540

Conversation

@john9x
Copy link
Copy Markdown
Contributor

@john9x john9x commented Jun 20, 2025

No description provided.

@john9x
Copy link
Copy Markdown
Contributor Author

john9x commented Jun 20, 2025

Ouch! I miss the check if (state.getClientSessionExpiryInterval() == 0) in the org.apache.activemq.artemis.core.protocol.mqtt.MQTTSession#stop

So it seems the problem in the documentation only and current code is fine.

https://github.com/apache/activemq-artemis/blob/37fbbc4c11aaa1ad30aaa5610aa71b9209e82bf8/docs/user-manual/mqtt.adoc?plain=1#L209

https://github.com/apache/activemq-artemis/blob/37fbbc4c11aaa1ad30aaa5610aa71b9209e82bf8/docs/user-manual/mqtt.adoc?plain=1#L212

@jbertram jbertram changed the title ARTEMIS-5540 consistent applying of defaultMqttSessionExpiryInterval ARTEMIS-5540 clarify doc for MQTT default session expiry interval Jun 20, 2025
@jbertram jbertram merged commit a80d016 into apache:main Jun 20, 2025
6 checks passed
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.

2 participants