fix(configuration): replace string literals with constants in error

vitshev · milosgajdos · web-flow · commit 96c9a85b62ee · 2024-12-16T22:30:37.000+01:00
Co-authored-by: Milos Gajdos &lt;milosgajdos83@gmail.com&gt;
Signed-off-by: Vitshev &lt;vitshev@tracto.ai&gt;
diff --git a/configuration/configuration.go b/configuration/configuration.go
@@ -928,7 +928,7 @@ func (clientAuth *ClientAuth) UnmarshalYAML(unmarshal func(interface{}) error) e
 	case ClientAuthVerifyClientCertIfGiven:
 	case ClientAuthRequireAndVerifyClientCert:
 	default:
-		return fmt.Errorf("invalid ClientAuth %s Must be one of: request-client-cert, require-any-client-cert, verify-client-cert-if-given, require-and-verify-client-cert", clientAuthString)
+		return fmt.Errorf("invalid ClientAuth %s Must be one of: %s, %s, %s, %s", clientAuthString, ClientAuthRequestClientCert, ClientAuthRequireAnyClientCert, ClientAuthVerifyClientCertIfGiven, ClientAuthRequireAndVerifyClientCert)
 	}
 
 	*clientAuth = ClientAuth(clientAuthString)

Original file line number	Diff line number	Diff line change
`@@ -928,7 +928,7 @@ func (clientAuth *ClientAuth) UnmarshalYAML(unmarshal func(interface{}) error) e`
`928`	`928`	`case ClientAuthVerifyClientCertIfGiven:`
`929`	`929`	`case ClientAuthRequireAndVerifyClientCert:`
`930`	`930`	`default:`
`931`		`- return fmt.Errorf("invalid ClientAuth %s Must be one of: request-client-cert, require-any-client-cert, verify-client-cert-if-given, require-and-verify-client-cert", clientAuthString)`
	`931`	`+ return fmt.Errorf("invalid ClientAuth %s Must be one of: %s, %s, %s, %s", clientAuthString, ClientAuthRequestClientCert, ClientAuthRequireAnyClientCert, ClientAuthVerifyClientCertIfGiven, ClientAuthRequireAndVerifyClientCert)`
`932`	`932`	`}`
`933`	`933`
`934`	`934`	`*clientAuth = ClientAuth(clientAuthString)`