io.vertx.ext.stomp.StompServerHandler#closeHandler is called twice

[shenweijiekdel]

Questions

Do not use this issue tracker to ask questions, instead use one of these channels. Questions will likely be closed without notice.

Version

4.5.8

Context

I am using a STOMP server over WebSocket.
When I actively close a connection, the closeHandler is called twice. this is the implementation of StompServerWebSocketConnectionImpl#close

vertx-stomp/src/main/java/io/vertx/ext/stomp/impl/StompServerWebSocketConnectionImpl.java

Lines 62 to 71 in 5522779

	@Override
	public void close() {
	cancelHeartbeat();
	handler().onClose(this);
	try {
	socket.close();
	} catch (IllegalStateException e) {
	// Ignore it, the web socket has already been closed.
	}
	}

it called handler and closed socket.

vertx-stomp/src/main/java/io/vertx/ext/stomp/impl/StompServerImpl.java

Line 130 in 5522779

socket.endHandler(v -> connection.close());

but called again by endHandler when websocket close frame was received

[tsegismont]

Thanks for your report, would you like to contribute a fix?

[shenweijiekdel]

This PR #95