search

davecheney / httpstat

It's like curl -v, with colours.
MIT License
6.96k stars 382 forks source link

Use TLS server name indication #39

Closed andres-erbsen closed 7 years ago

andres-erbsen commented 7 years ago

https://github.com/davecheney/httpstat/blob/master/main.go#L134 should set ServerName in tls.Config.

This would make it possible to use this tool with hosting situations where multiple sites share the same IP and there is no fallback.

davecheney commented 7 years ago

Yes, this is an important improvement. I think this behaviour should be on by default, see #40 for a suggestion for a flag to disable it.

theckman commented 7 years ago

Looks like this may have been closed by #42?

davecheney commented 7 years ago

Well spotted.