Skip to content

Support using SNI from the command line #440

Closed
wants to merge 3 commits into from

4 participants

@Rush
Rush commented Jul 4, 2013

This resolves issue #401.

@Rush
Rush commented Sep 9, 2013

I wonder, is anything in particular stopping this pull request? These are merely improvements to the node-http-proxy command line tool.

@Nowaker
Nowaker commented Sep 9, 2013

+1 for merging this. CA bundles are required for some SSL providers, e.g. certum.pl.

@indexzero
nodejitsu member

No tests. Fails JSHint.

@indexzero indexzero closed this Dec 27, 2013
@Rush
Rush commented Dec 27, 2013

Is there any will to have this? As I understand you are stripping down node-http-proxy to bare proxying library. (which is good!!) Since this pull request I have made a more high level proxy with above feature at https://github.com/CodeCharmLtd/rush-http-proxy currently being improved to use caronte and have its own http-dispatch-table with host-based / wildcard and regexp.

@yawnt
yawnt commented Dec 27, 2013

No, the caronte branch is going to be a library only. But thanks for asking :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.