fabiolb / fabio

Consul Load-Balancing made simple
https://fabiolb.net
MIT License
7.25k stars 619 forks source link

https+tcp+sni listener support #784

Closed nathanejohnson closed 4 years ago

nathanejohnson commented 4 years ago

Adds support for the https+tcp+sni listener. Any host routes that are marked with proto=tcp or have a scheme tcp:// will be matched for TCP SNI steering. Failing any matches there, fallthrough will be to https matching. This resolves: #783.

clean up error handling on 'use of closed network connection' case

fix makefile tag finding stuff

fix logic for https+tcp+sni matching so that only explicit proto=tcp matches

nathanejohnson commented 4 years ago

I'll give this a few days, failing any negative reviews I'm merging this. So deadline is September 2 to voice concerns