Make Let's Encrypt module honor the Bind settings

This commit is contained in:
Ben Schumacher 2016-01-03 00:37:57 -07:00
parent b6326d402d
commit 82b0c0b9eb
3 changed files with 10 additions and 5 deletions

View File

@ -42,8 +42,8 @@ func init() {
var directiveOrder = []directive{
// Essential directives that initialize vital configuration settings
{"root", setup.Root},
{"tls", setup.TLS}, // letsencrypt is set up just after tls
{"bind", setup.BindHost},
{"tls", setup.TLS}, // letsencrypt is set up just after tls
// Other directives that don't create HTTP handlers
{"startup", setup.Startup},

View File

@ -445,8 +445,9 @@ func redirPlaintextHost(cfg server.Config) server.Config {
}
return server.Config{
Host: cfg.Host,
Port: "http",
Host: cfg.Host,
BindHost: cfg.BindHost,
Port: "http",
Middleware: map[string][]middleware.Middleware{
"/": []middleware.Middleware{redirMidware},
},

View File

@ -38,14 +38,18 @@ func TestHostQualifies(t *testing.T) {
func TestRedirPlaintextHost(t *testing.T) {
cfg := redirPlaintextHost(server.Config{
Host: "example.com",
Port: "http",
Host: "example.com",
BindHost: "93.184.216.34",
Port: "http",
})
// Check host and port
if actual, expected := cfg.Host, "example.com"; actual != expected {
t.Errorf("Expected redir config to have host %s but got %s", expected, actual)
}
if actual, expected := cfg.BindHost, "93.184.216.34"; actual != expected {
t.Errorf("Expected redir config to have bindhost %s but got %s", expected, actual)
}
if actual, expected := cfg.Port, "http"; actual != expected {
t.Errorf("Expected redir config to have port '%s' but got '%s'", expected, actual)
}