fix(server): stricter headers security check

[knagaitsev]

• This is a bugfix
• This is a feature
• This is a code refactor
• This is a test update
• This is a docs update
• This is a metadata update

For Bugs and Features; did you add new tests?

Yes

Motivation / Use-Case

Headers security check for connecting clients should not allow a connection without headers: #2077

Breaking Changes

Changes the socket server implementation requirements such that headers must be passed to onConnection callback.

Additional Info

[codecov]

Codecov Report

Merging #2092 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2092      +/-   ##
==========================================
+ Coverage   94.45%   94.46%   +<.01%     
==========================================
  Files          32       32              
  Lines        1208     1210       +2     
  Branches      333      334       +1     
==========================================
+ Hits         1141     1143       +2     
  Misses         65       65              
  Partials        2        2

Impacted Files	Coverage Δ
lib/servers/WebsocketServer.js	94.11% <ø> (ø)	⬆️
lib/servers/SockJSServer.js	96.66% <ø> (ø)	⬆️
lib/Server.js	96.93% <100%> (+0.01%)	⬆️

---

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 56274e4...e336915. Read the comment docs.

[alexander-akait]

Thanks for clarify and good job!