diff options
author | Christopher Speller <crspeller@gmail.com> | 2018-02-23 15:33:36 -0800 |
---|---|---|
committer | Derrick Anderson <derrick@andersonwebstudio.com> | 2018-02-23 18:33:36 -0500 |
commit | 838295f82c4bad22bd83a30a5b157dd717f8ceb6 (patch) | |
tree | 5000f9bb3f49f434848a300acb27abbaa7f8d90d /app/server.go | |
parent | babccba8f88f58c50c345cc770550706df2f04c0 (diff) | |
download | chat-838295f82c4bad22bd83a30a5b157dd717f8ceb6.tar.gz chat-838295f82c4bad22bd83a30a5b157dd717f8ceb6.tar.bz2 chat-838295f82c4bad22bd83a30a5b157dd717f8ceb6.zip |
Fix forward 80 to 443 to not bind on all interfaces. (#8360)
Diffstat (limited to 'app/server.go')
-rw-r--r-- | app/server.go | 29 |
1 files changed, 17 insertions, 12 deletions
diff --git a/app/server.go b/app/server.go index afa282ad6..93804a372 100644 --- a/app/server.go +++ b/app/server.go @@ -171,20 +171,25 @@ func (a *App) StartServer() error { } if *a.Config().ServiceSettings.Forward80To443 { - if *a.Config().ServiceSettings.UseLetsEncrypt { - go http.ListenAndServe(":http", m.HTTPHandler(nil)) + if host, _, err := net.SplitHostPort(addr); err != nil { + l4g.Error("Unable to setup forwarding: " + err.Error()) } else { - go func() { - redirectListener, err := net.Listen("tcp", ":80") - if err != nil { - listener.Close() - l4g.Error("Unable to setup forwarding: " + err.Error()) - return - } - defer redirectListener.Close() + httpListenAddress := net.JoinHostPort(host, "http") - http.Serve(redirectListener, http.HandlerFunc(redirectHTTPToHTTPS)) - }() + if *a.Config().ServiceSettings.UseLetsEncrypt { + go http.ListenAndServe(httpListenAddress, m.HTTPHandler(nil)) + } else { + go func() { + redirectListener, err := net.Listen("tcp", httpListenAddress) + if err != nil { + l4g.Error("Unable to setup forwarding: " + err.Error()) + return + } + defer redirectListener.Close() + + http.Serve(redirectListener, http.HandlerFunc(redirectHTTPToHTTPS)) + }() + } } } |