Skip to content
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.

Sign up for GitHub

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

Jump to bottom

E12/S1: Configurable TLS and SNI support for multiple certs/domains #13

Closed
jimhw opened this issue Apr 10, 2017 · 3 comments
Closed

E12/S1: Configurable TLS and SNI support for multiple certs/domains #13

jimhw opened this issue Apr 10, 2017 · 3 comments
Assignees
@DrMegavolt

Comments

@jimhw
Copy link

jimhw commented Apr 10, 2017 •
edited

Allow TLS and SNI support for https to be configurable externally so that multiple certs can be used for multiple domains.
@jimhw jimhw changed the title Support https Support for https Apr 10, 2017
@jimhw jimhw added ready and removed ready labels Apr 10, 2017
@jimhw jimhw added the ready label Apr 24, 2017
@jimhw jimhw changed the title Support for https Configurable TLS and SNI support for multiple certs/domains Apr 24, 2017
@jimhw jimhw added this to the Phase 1 milestone Apr 24, 2017
@jimhw jimhw added ready and removed ready labels Apr 24, 2017
@jimhw jimhw changed the title Configurable TLS and SNI support for multiple certs/domains E12/S1: Configurable TLS and SNI support for multiple certs/domains Apr 24, 2017
@altsang altsang removed the ready label Apr 25, 2017
@jimhw jimhw added the ready label Apr 27, 2017
@altsang
Copy link
Contributor

altsang commented Apr 28, 2017

this functionality is already there in EG per what Roman built right?

https://github.com/LunchBadger/express-gateway#tls

@DrMegavolt
Copy link
Contributor

we need to start both http + https servers at the same time, not single one.

@jimhw jimhw removed the ready label May 9, 2017
DrMegavolt pushed a commit that referenced this issue May 12, 2017
refactored server loader; still start only one of them tbd in #13
8f56873
@DrMegavolt DrMegavolt mentioned this issue May 12, 2017
Merged
DrMegavolt added a commit that referenced this issue May 14, 2017
@DrMegavolt
Feature/config loader restructure; closes #88 (#91)
2f3d396 
* moving config to separate folder for further separation

* refactored server loader; still start only one of them tbd in #13

* refactored pipelines loader

* refactored file config loader; uses js-yaml instead of JSON.parse; supports both formats
@altsang
Copy link
Contributor

altsang commented May 23, 2017

mirror image of the server exposed as https: and http:

IrfanBaqui pushed a commit that referenced this issue May 25, 2017
@DrMegavolt @IrfanBaqui
Feature/https http server in parallel connect #13 connect #102 (#123)
067e94b 
Configure gateway to listen on http and https ports at the same time, and add TLS tests to ensure SNI callback works as expected.
@IrfanBaqui IrfanBaqui removed the test label May 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DrMegavolt DrMegavolt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jimhw @DrMegavolt @altsang @IrfanBaqui