caddy/caddytest
Francis Lavoie a58f240d3e
httpcaddyfile: Fix (auto-HTTPS edgecase) ()
Guh, this is complicated.

Fixes 

This also follows up on  (reverting it) which made a change that technically worked, but was incorrect. It changed the condition in `hostsFromKeysNotHTTP` from `&&` to `||`, but then the function no longer did what its name said it would do, and it would return hosts even if they were marked with `http://`, if they used a non-HTTP port. That wasn't the intent of it. The test added in there was kept though, because it is a valid usecase.

The actual fix is to check _earlier_ whether all the addresses explicitly have `http://`, and if so we can short circuit and skip considering the rest.
2022-03-24 22:54:03 -06:00
..
integration httpcaddyfile: Fix (auto-HTTPS edgecase) () 2022-03-24 22:54:03 -06:00
a.caddy.localhost.crt caddyhttp: Add default SNI tests () 2020-03-17 12:39:01 -06:00
a.caddy.localhost.key caddyhttp: Add default SNI tests () 2020-03-17 12:39:01 -06:00
caddy.ca.cer httpcaddyfile: Add client_auth options to tls directive () 2020-06-05 12:19:36 -06:00
caddy.localhost.crt caddyhttp: Fix default SNI for default conn policy () 2020-03-13 11:32:53 -06:00
caddy.localhost.key caddyhttp: Fix default SNI for default conn policy () 2020-03-13 11:32:53 -06:00
caddytest.go Move from deprecated ioutil to os and io packages () 2021-09-29 11:17:48 -06:00
caddytest_test.go caddyhttp: Fix default SNI for default conn policy () 2020-03-13 11:32:53 -06:00