[Bug]: Broker URL path is dropped in `connect()`, connections fail

[dmotz]

MQTTjs Version

5.6.0

Broker

Eclipse, Hive, MQTTHQ, EMQX

Environment

Browser

Description

A release after 5.3.5 strips paths from broker URLs.

For example, the /mqtt in these broker URLs is dropped and the socket connection is made to the root path /:
wss://mqtt.eclipseprojects.io/mqtt
wss://broker.emqx.io:8084/mqtt
wss://broker.hivemq.com:8884/mqtt
wss://public.mqtthq.com:8084/mqtt

These connections fail as the path is needed.

Minimal Reproduction

mqtt.connect(url) where url contains a path, e.g.:

wss://mqtt.eclipseprojects.io/mqtt
wss://broker.emqx.io:8084/mqtt
wss://broker.hivemq.com:8884/mqtt
wss://public.mqtthq.com:8084/mqtt

Debug logs

mqtt.esm.js:5 WebSocket connection to 'wss://mqtt.eclipseprojects.io/' failed

[erich2s]

Same here, and I downgraded to 5.5.5, works fine

[robertsLando]

I have submitted a PR to fix the issue but there is a discussion going on about this if you would like to join so we find a definitive solution to the issue

[robertsLando]

5.7.0 should fix the issue