mirror of
https://github.com/soheilhy/cmux.git
synced 2025-09-17 12:10:08 +08:00
Merge pull request #2 from tamird/update-readme-reasons
README: clarify why TLS is limited
This commit is contained in:
@@ -57,8 +57,9 @@ is negligible.
|
|||||||
*TODO(soheil)*: Add benchmarks.
|
*TODO(soheil)*: Add benchmarks.
|
||||||
|
|
||||||
## Limitations
|
## Limitations
|
||||||
* *TLS*: Since `cmux` sits in between the actual listener and the mux'ed
|
* *TLS*: `net/http` uses a type assertion to identify TLS connections; since
|
||||||
listeners, TLS handshake is not handled inside the actual servers.
|
cmux's lookahead-implementing connection wraps the underlying TLS connection,
|
||||||
|
this type assertion fails.
|
||||||
Because of that, you can serve HTTPS using cmux but `http.Request.TLS`
|
Because of that, you can serve HTTPS using cmux but `http.Request.TLS`
|
||||||
would not be set in your handlers.
|
would not be set in your handlers.
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user