klawr / mqtt-inspector

MIT License
2 stars 1 forks source link

Remove necessity for port 8080 #1

Closed klawr closed 7 months ago

klawr commented 7 months ago

Problem Description At the moment two ports have to be opened. 3030 for the webserver and 8080 for the websocket connection.

Proposed Solution Use only one port to handle both browser get requests and websocket. For this we can utilize https://developer.mozilla.org/en-US/docs/Web/HTTP/Protocol_upgrade_mechanism Not sure yet if warp should be replaced by some tokio_tungstenite method or vice versa. Both presumably support simple requests and websockets. Suggestion is to only use port 3030

klawr commented 7 months ago

Fixed with 8ed7a6beff2ec9c02d93b964985bf997708b3552