Merge pull request #103 from diegopereiraeng/main

sonar_token mandatory field lock

Author: diegopereiraeng

README.md

@@ -6,7 +6,7 @@ This plugin is designed to run SonarQube scans, handle the results, and convert
 
 - **New Parameter: `sonar_config_file`**
   - **Type**: Boolean
-  - **Description**: If set to true, the plugin will utilize the specified `sonar-project.properties` file for the SonarQube analysis, if it exists.
+  - **Description**: If set to true, the plugin will utilize the specified `sonar-project.properties` file for the SonarQube analysis, if it exists. sonar_token param still mandatory.
   - **Environment Variable**: `PLUGIN_SONAR_CONFIG_FILE`
   - **Allowed Values**: `"true"`, `"false"`
 

plugin.go

@@ -505,9 +505,14 @@ func (p Plugin) Exec() error {
 			args = append(args, "-Dsonar.host.url="+p.Config.Host)
 		}
 
-		if len(p.Config.Token) >= 1 && p.Config.UseSonarConfigFileOverride {
+		if len(p.Config.Token) >= 1 {
 			fmt.Println("OVERRIDING sonar.login=" + p.Config.Token)
 			args = append(args, "-Dsonar.login="+p.Config.Token)
+		} else {
+			fmt.Println("sonar_token param is also mandatory when using sonar-project.properties.")
+			fmt.Println("Exiting with status 2")
+			os.Exit(2)
+
 		}
 
 		if len(p.Config.Key) >= 1 && p.Config.UseSonarConfigFileOverride {