Bunch of changes for CloseReason class
- Add final modifier for fields
- UTF-8 encoded bytes for reasonPhrase cannot be greater than 123 bytes. Throw IllegalArgumentException in that case.
- enum constants for status codes need be documented. Otherwise, javadoc would be empty
Looks like cannot attach the patch. Will send by email.
If a close frame doesn't include status code, the Endpoint#onClose() is invoked with a CloseReason. What would be the value of CloseReason#getCloseCode() return in that case ? Would it be 1005 ? Good to include that in the spec.