-
Notifications
You must be signed in to change notification settings - Fork 288
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HTTP+HTTPS server simultaneously #191
Comments
I'm happy to have multiple listeners. I was unhappy with the previous patch set because I thought it was too complex and not re-usable enough. Do you need to distinguish HTTP from HTTPS traffic inside the application? |
I think, it will be enought to:
|
I'm thinking something like
|
Yes, this or something like this will be enough for 99% cases. |
Evan, maybe something like protocol() -> http | https --Kai On Fri, Jan 25, 2013 at 9:10 AM, Evan Miller notifications@github.comwrote:
|
@kotedo, good call since SPDY is around the corner. |
Ok, I submitted a pull request to SimpleBridge, check out |
Btw, i pushed some code to SimpleBridge for support of X-Forwarded-Proto header ( nitrogen/simple_bridge#32 ). It can be useful when nginx is in charge of SSL, and cb is running behind. |
At time of writing this comment, NGINX working fine with CB for all protocols it supports including SPDY. Also, there is ways to distinguish http and https traffic. And for all webservers' related stuff we rely on simple_bridge, so that feature should be implemented there, may be as part of nitrogen/simple_bridge#35 |
It will be good to listen HTTP and HTTPS ports simultaneously. Current easy-to-use variant,
CB
+nginx
as proxy, crashes on chunked encoding and otherHTTP/1.1
features. This is because nginx does not supportHTTP/1.1
.By now, other possible crunches are:
CB
+apache
+mod_proxy
+mod_ssl
- it works! But have much more complexity (compared withnginx
).CB
in cluster: disadvantages are overhead of session replication, double RAM consumption and overall system design looks ugly.Also, a patch was proposed some time ago.
Hope, this will be implemented in future.
Original message from google-groups.
Some related patches was in pull-request #44 "Multiple listeners".
The text was updated successfully, but these errors were encountered: