Mirrors/coredns
1
0
Fork 0
mirror of https://github.com/coredns/coredns.git synced 2025-11-01 02:33:14 -04:00
Code Issues Packages Projects Releases Wiki Activity

mw/autopath: correct type for type switch (#1029)

Browse Source 
* mw/autopath: correct type for type switch

Use pointer to kubernetes as that is what is registered. Fix up
federation which had the same mistake.

* here
This commit is contained in:
Miek Gieben
2017-09-06 23:28:41 +01:00
committed by Chris O'Haver
parent c514197d6b
commit 9528777fc5
2 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
middleware/autopath/setup.go
View File

@@ -28,12 +28,11 @@ func setup(c *caddy.Controller) error {
// Do this in OnStartup, so all middleware has been initialized. // Do this in OnStartup, so all middleware has been initialized.
c.OnStartup(func() error { c.OnStartup(func() error {
// TODO(miek): fabricate test to proof this is not thread safe.
m := dnsserver.GetConfig(c).Handler(mw) m := dnsserver.GetConfig(c).Handler(mw)
if m == nil { if m == nil {
return nil return nil
} }
if x, ok := m.(kubernetes.Kubernetes); ok { if x, ok := m.(*kubernetes.Kubernetes); ok {
ap.searchFunc = x.AutoPath ap.searchFunc = x.AutoPath
} }
if x, ok := m.(*erratic.Erratic); ok { if x, ok := m.(*erratic.Erratic); ok {

4
middleware/federation/setup.go
View File

@@ -30,7 +30,7 @@ func setup(c *caddy.Controller) error {
if m == nil { if m == nil {
return nil return nil
} }
if x, ok := m.(kubernetes.Kubernetes); ok { if x, ok := m.(*kubernetes.Kubernetes); ok {
fed.Federations = x.Federations fed.Federations = x.Federations
} }
return nil return nil
@@ -38,7 +38,7 @@ func setup(c *caddy.Controller) error {
dnsserver.GetConfig(c).AddMiddleware(func(next middleware.Handler) middleware.Handler { dnsserver.GetConfig(c).AddMiddleware(func(next middleware.Handler) middleware.Handler {
fed.Next = next fed.Next = next
return nil return fed
}) })
return nil return nil