#533 drafts receiveCloseStatus method

[OlegDokuka]

This PR provides receiveCloseStatus for WebSocketInbound

[codecov-io]

Codecov Report

Merging #815 into master will increase coverage by 0.08%.
The diff coverage is 95.83%.

@@             Coverage Diff              @@
##             master     #815      +/-   ##
============================================
+ Coverage     67.07%   67.16%   +0.08%     
- Complexity     1436     1440       +4     
============================================
  Files           137      137              
  Lines          6725     6743      +18     
  Branches        864      864              
============================================
+ Hits           4511     4529      +18     
+ Misses         1756     1755       -1     
- Partials        458      459       +1

Impacted Files	Coverage Δ	Complexity Δ
...reactor/netty/http/websocket/WebsocketInbound.java	100% <ø> (ø)	3 <0> (ø)	⬇️
...r/netty/http/client/WebsocketClientOperations.java	72.82% <100%> (+3.77%)	22 <1> (+2)	⬆️
...r/netty/http/server/WebsocketServerOperations.java	66.3% <92.85%> (+2.88%)	22 <3> (+3)	⬆️
...eactor/netty/http/client/HttpClientOperations.java	64.22% <0%> (-0.88%)	71% <0%> (-1%)
...c/main/java/reactor/netty/channel/FluxReceive.java	77.5% <0%> (ø)	62% <0%> (ø)	⬇️
...ctor/netty/resources/PooledConnectionProvider.java	81.88% <0%> (+0.78%)	25% <0%> (ø)	⬇️
.../java/reactor/netty/channel/ChannelOperations.java	92.74% <0%> (+0.8%)	59% <0%> (ø)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5ab4470...51a897d. Read the comment docs.

[violetagg]

@OlegDokuka @smaldini I think that the event should be emitted when the close frame is sent i.e.

For the client:
https://github.com/reactor/reactor-netty/blob/master/src/main/java/reactor/netty/http/client/WebsocketClientOperations.java#L206
https://github.com/reactor/reactor-netty/blob/master/src/main/java/reactor/netty/http/client/WebsocketClientOperations.java#L223

For the server:
https://github.com/reactor/reactor-netty/blob/master/src/main/java/reactor/netty/http/server/WebsocketServerOperations.java#L193
https://github.com/reactor/reactor-netty/blob/master/src/main/java/reactor/netty/http/server/WebsocketServerOperations.java#L211

[smaldini]

I think the event should be sent in these 4 places before the writeAndFlush in the protected CLOSE_SENT.getAndSet(this, 1) == 0) path.

[violetagg]

Thanks

[violetagg]

I changed the title of the commit with this 78950ac