search

mesosphere / traefik-forward-auth

219 stars 47 forks source link

fix invalid error log message #27

Closed foosinn closed 4 years ago

foosinn commented 4 years ago

a run of go test ./... failed for me due to an invalid Errorf statement.

k3a commented 4 years ago

You should probably use %v to make it consistent with other errors in the repo. There doesn't seem to be difference now (https://play.golang.org/p/01OEs7caVsA) but I think there used to be difference in previous Go versions.

foosinn commented 4 years ago

Updated to use %v

mhrabovcin commented 4 years ago

@jr0d can we merge this one?

jr0d commented 4 years ago

Thanks @foosinn