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

[Bug] Auto-Reconnect does not handle configuration change #1746

Open
SpeckiJ opened this issue Sep 2, 2024 · 1 comment
Open

[Bug] Auto-Reconnect does not handle configuration change #1746

SpeckiJ opened this issue Sep 2, 2024 · 1 comment
Assignees
Labels
bug Something isn't working web MQTTX Web
Milestone

Comments

@SpeckiJ
Copy link

SpeckiJ commented Sep 2, 2024

What did I do

  1. Try to connect to a broker with default settings but invalid Path (e.g. connecting to wss://broker.emqx.io:8084/mqtt_invalid_path.

  2. Notice that the Path is wrong, open configuration and fix the path (/mqtt_invalid_path --> /mqtt).

  3. Close configuration and start a new connection with the new configuration

What happened

The client successfully connects after fixing the configuration. But I receive Alerts about Reconnecting every couple of seconds. When looking at the Network-Tab in the Browser console it shows that it still tries to reconnect with the old invalid configuration, although i have changed them already.

The only workaround i have found so far to get rid of the background-reconnects is to refresh the page (via F5 or Ctrl+R in the Desktop Client).

Expected

When changing the configuration of a client all auto-reconnection attempts for this client should be cancelled. After I have changed the configuration the old configuration should not be used anymore.

Environment

More detail

Screenshot of Proxy showing the requests that are sent out by the client. Blue marks the point in time where the configuration was fixed:

mqttx_test drawio(1)

This is probably a duplicate to #1680 that was unfortunately closed due to inactivity.

@SpeckiJ SpeckiJ added the bug Something isn't working label Sep 2, 2024
@ysfscream ysfscream self-assigned this Sep 3, 2024
@ysfscream ysfscream added the web MQTTX Web label Sep 3, 2024
@ysfscream ysfscream added this to the v1.11.0 milestone Sep 3, 2024
@ysfscream
Copy link
Member

@SpeckiJ Hi, this is indeed a BUG that appeared on Desktop before and might not have been completely fixed in the Web version. Thank you for your feedback, we will fix it in the next version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working web MQTTX Web
Projects
Status: Backlog
Development

No branches or pull requests

2 participants