Commit 287e2801 authored by Iñigo's avatar Iñigo Committed by Miek Gieben

plugin/grpc: bugfix setup parser (#2697)

Fix setup parser. Config params are not being setting.
parent 7b6cb762
...@@ -81,6 +81,12 @@ func parseGRPCStanza(c *caddyfile.Dispenser) (*GRPC, error) { ...@@ -81,6 +81,12 @@ func parseGRPCStanza(c *caddyfile.Dispenser) (*GRPC, error) {
return g, err return g, err
} }
for c.NextBlock() {
if err := parseBlock(c, g); err != nil {
return g, err
}
}
if g.tlsServerName != "" { if g.tlsServerName != "" {
if g.tlsConfig == nil { if g.tlsConfig == nil {
g.tlsConfig = new(tls.Config) g.tlsConfig = new(tls.Config)
...@@ -95,12 +101,6 @@ func parseGRPCStanza(c *caddyfile.Dispenser) (*GRPC, error) { ...@@ -95,12 +101,6 @@ func parseGRPCStanza(c *caddyfile.Dispenser) (*GRPC, error) {
g.proxies = append(g.proxies, pr) g.proxies = append(g.proxies, pr)
} }
for c.NextBlock() {
if err := parseBlock(c, g); err != nil {
return g, err
}
}
return g, nil return g, nil
} }
......
...@@ -74,9 +74,6 @@ func TestSetupTLS(t *testing.T) { ...@@ -74,9 +74,6 @@ func TestSetupTLS(t *testing.T) {
tls_servername dns tls_servername dns
}`, false, "dns", ""}, }`, false, "dns", ""},
{`grpc . 127.0.0.1 { {`grpc . 127.0.0.1 {
tls_servername dns
}`, false, "", ""},
{`grpc . 127.0.0.1 {
tls tls
}`, false, "", ""}, }`, false, "", ""},
{`grpc . 127.0.0.1`, false, "", ""}, {`grpc . 127.0.0.1`, false, "", ""},
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment