Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modification of Session Expiry Interval in DISCONNECT does not work #1833

Closed
ckrey opened this issue Sep 18, 2018 · 2 comments
Assignees
Labels
Milestone

Comments

@ckrey
Copy link

@ckrey ckrey commented Sep 18, 2018

Environment

  • OS: Debian 9
  • Erlang/OTP: 19
  • EMQ: 3.0 beta 2

Description

CONNECT with a Session Expiry Interval of 3 seconds
DISCONNECT with setting a new Session Expiry Interval of 30 seconds
Wait 10 seconds
Re-CONNECT with CleanStart = false

emqx does not indicate existing session, but should do so

19:32:22.471 [debug] MQTT(10.0.2.2:58906): RECV <<16,83,0,4,77,81,84,84,5,2,0,60,5,17,0,0,0,3,0,65,45,91,77,81,84,84,118,53,83,101,115,115,105,111,110,69,120,112,105,114,121,32,116,101,115,116,95,115,69,73,95,115,109,97,108,108,95,100,105,115,99,111,110,110,101,99,116,95,108,97,114,103,101,114,95,114,101,99,111,110,110,101,99,116,93>>
19:32:22.472 [debug] MQTT(@10.0.2.2:58906): RECV CONNECT(Q0, R0, D0, ClientId=-[MQTTv5SessionExpiry test_sEI_small_disconnect_larger_reconnect], ProtoName=MQTT, ProtoVsn=5, CleanStart=true, KeepAlive=60, Username=undefined, Password=undefined)
19:32:22.474 [debug] MQTT(-[MQTTv5SessionExpiry test_sEI_small_disconnect_larger_reconnect]@10.0.2.2:58906): SEND CONNACK(Q0, R0, D0, AckFlags=0, ReasonCode=0)
19:32:22.475 [debug] MQTT(10.0.2.2:58906): SEND <<32,21,0,0,18,39,0,16,0,0,36,2,37,1,42,1,41,1,34,0,0,40,1>>
19:32:22.476 [debug] MQTT(10.0.2.2:58906): Keepalive at the interval of 45
19:32:23.477 [debug] MQTT(10.0.2.2:58906): RECV <<224,7,0,5,17,0,0,0,30>>
19:32:23.477 [debug] MQTT(-[MQTTv5SessionExpiry test_sEI_small_disconnect_larger_reconnect]@10.0.2.2:58906): RECV DISCONNECT(Q0, R0, D0, ReasonCode=0)
19:32:23.478 [debug] MQTT(10.0.2.2:58906): Terminated for normal
19:32:23.479 [info] MQTT(-[MQTTv5SessionExpiry test_sEI_small_disconnect_larger_reconnect]@10.0.2.2:58906): Shutdown for normal
19:32:34.036 [debug] MQTT(10.0.2.2:58908): RECV <<16,78,0,4,77,81,84,84,5,0,0,60,0,0,65,45,91,77,81,84,84,118,53,83,101,115,115,105,111,110,69,120,112,105,114,121,32,116,101,115,116,95,115,69,73,95,115,109,97,108,108,95,100,105,115,99,111,110,110,101,99,116,95,108,97,114,103,101,114,95,114,101,99,111,110,110,101,99,116,93>>
19:32:34.036 [debug] MQTT(@10.0.2.2:58908): RECV CONNECT(Q0, R0, D0, ClientId=-[MQTTv5SessionExpiry test_sEI_small_disconnect_larger_reconnect], ProtoName=MQTT, ProtoVsn=5, CleanStart=false, KeepAlive=60, Username=undefined, Password=undefined)
19:32:34.038 [debug] MQTT(-[MQTTv5SessionExpiry test_sEI_small_disconnect_larger_reconnect]@10.0.2.2:58908): SEND CONNACK(Q0, R0, D0, AckFlags=0, ReasonCode=0)
19:32:34.039 [debug] MQTT(10.0.2.2:58908): SEND <<32,21,0,0,18,39,0,16,0,0,36,2,37,1,42,1,41,1,34,0,0,40,1>>
19:32:34.039 [debug] MQTT(10.0.2.2:58908): Keepalive at the interval of 45
19:32:35.043 [debug] MQTT(10.0.2.2:58908): RECV <<224,0>>
19:32:35.043 [debug] MQTT(-[MQTTv5SessionExpiry test_sEI_small_disconnect_larger_reconnect]@10.0.2.2:58908): RECV DISCONNECT(Q0, R0, D0, ReasonCode=0)
19:32:35.043 [debug] MQTT(10.0.2.2:58908): Terminated for normal
19:32:35.044 [info] MQTT(-[MQTTv5SessionExpiry test_sEI_small_disconnect_larger_reconnect]@10.0.2.2:58908): Shutdown for normal
@ckrey ckrey changed the title Modification of Session Expiry Interval in DISCONNECT does not always work Modification of Session Expiry Interval in DISCONNECT does not work Sep 18, 2018
@turtleDeng turtleDeng added this to the 3.0-beta.3 milestone Sep 18, 2018
@gilbertwong96

This comment has been minimized.

Copy link
Contributor

@gilbertwong96 gilbertwong96 commented Sep 19, 2018

It has not been not implemented, same as #1834

@tigercl

This comment has been minimized.

Copy link
Collaborator

@tigercl tigercl commented Sep 26, 2018

hi, @ckrey, it has been fixed in #1845

@tigercl tigercl closed this Sep 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.