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!