fix: validate scm auth info problem (#1502)

only check token type auth, also need to check password type auth.

Co-authored-by: zhujian <zhujian@caicloud.io>

pkg/server/biz/scm/scm.go

@@ -130,16 +130,21 @@ func GenerateSCMToken(config *v1alpha1.SCMSource) error {
 
 	if generatedToken != "" && config.AuthType == v1alpha1.AuthTypePassword {
 		config.Token = generatedToken
-	} else {
-		err := provider.CheckToken()
-		if err != nil {
-			return err
-		}
 	}
 
 	// Cleanup the password for security.
 	config.Password = ""
 
+	// recreate the scm provider and check auth
+	provider, err = GetSCMProvider(config)
+	if err != nil {
+		return err
+	}
+	err = provider.CheckToken()
+	if err != nil {
+		return err
+	}
+
 	return nil
 }
 

pkg/util/cerr/error.go

@@ -118,7 +118,7 @@ var (
 
 	// ErrorExternalAuthorizationFailed defines error that authorization failed for external system, Unauthorized 401.
 	ErrorExternalAuthorizationFailed = nerror.InternalServerError.Build(ReasonAuthorizationFailed,
-		"authentication failed: ${error}")
+		"authorization failed: ${error}")
 
 	// ErrorExternalAuthenticationFailed defines error that authentication failed for external system, Forbidden 403.
 	ErrorExternalAuthenticationFailed = nerror.InternalServerError.Build(ReasonAuthenticationFailed,