rm nil check

api/v1beta1/additional.go

@@ -143,12 +143,12 @@ type BasicAuth struct {
 	// for authentication.
 	// It must be at them same namespace as CRD
 	// +optional
-	Username *v1.SecretKeySelector `json:"username,omitempty"`
+	Username v1.SecretKeySelector `json:"username,omitempty"`
 	// The secret in the service scrape namespace that contains the password
 	// for authentication.
 	// It must be at them same namespace as CRD
 	// +optional
-	Password *v1.SecretKeySelector `json:"password,omitempty"`
+	Password v1.SecretKeySelector `json:"password,omitempty"`
 	// PasswordFile defines path to password file at disk
 	// +optional
 	PasswordFile string `json:"password_file,omitempty"`

controllers/factory/alertmanager/config.go

@@ -745,26 +745,22 @@ func (cb *configBuilder) buildHTTPConfig(httpCfg *operatorv1beta1.HTTPConfig) (y
 func (cb *configBuilder) buildBasicAuth(basicAuth *operatorv1beta1.BasicAuth) (yaml.MapSlice, error) {
 	var r yaml.MapSlice
 
-  if basicAuth.Username != nil {
-		u, err := cb.fetchSecretValue(&basicAuth.Username)
-		if err != nil {
-			return nil, err
-		}
-		r = append(r, yaml.MapItem{
-			Key:   "username",
-			Value: string(u),
-		})
-  }
-  if basicAuth.Password != nil {
-		p, err := cb.fetchSecretValue(&basicAuth.Password)
-		if err != nil {
-			return nil, err
-		}
-		r = append(r, yaml.MapItem{
-			Key:   "password",
-			Value: string(p),
-		})
-  }
+	u, err := cb.fetchSecretValue(&basicAuth.Username)
+	if err != nil {
+		return nil, err
+	}
+	r = append(r, yaml.MapItem{
+		Key:   "username",
+		Value: string(u),
+	})
+	p, err := cb.fetchSecretValue(&basicAuth.Password)
+	if err != nil {
+		return nil, err
+	}
+	r = append(r, yaml.MapItem{
+		Key:   "password",
+		Value: string(p),
+	})
   if len(basicAuth.PasswordFile) > 0 {
 		r = append(r, yaml.MapItem{
 			Key:   "password_file",