Merge pull request #20 from codacy/add_api_token_to_upload

Allow the usage of the API token to upload results

Author: heliocodacy

README.md

@@ -14,11 +14,19 @@ The `codacy-cli-v2` is a command-line tool for Codacy that supports analyzing co
     - `--fix, -f`: Automatically fixes issues when possible.
     - `--new-pr`: Creates a new GitHub PR with fixed issues.
 
-- **`upload` Command**: Uploads a SARIF file containing analysis results to Codacy.
+- **`upload` Command With Project Token**: Uploads a SARIF file containing analysis results to Codacy.
     - `--sarif-path, -s`: Path to the SARIF report.
     - `--commit-uuid, -c`: Commit UUID.
     - `--project-token, -t`: Project token for Codacy API.
 
+- **`upload` Command With API Token**: Uploads a SARIF file containing analysis results to Codacy.
+    - `--sarif-path, -s`: Path to the SARIF report.
+    - `--commit-uuid, -c`: Commit UUID.
+    - `--api-token, -a`: User level token for Codacy API.
+    - `--provider, -p`: Provider name (e.g., gh, gl, bb).
+    - `--owner, -o`: Repository owner.
+    - `--repository, -r`: Repository name.
+
 ### Important Concepts
 
 - **`.codacy/.codacy.yaml`**: Configuration file to specify `node` and `eslint` versions for the CLI.

cmd/analyze.go

@@ -1,7 +1,6 @@
 package cmd
 
 import (
-	"bytes"
 	"codacy/cli-v2/config"
 	"codacy/cli-v2/tools"
 	"codacy/cli-v2/utils"
@@ -12,8 +11,6 @@ import (
 	"net/http"
 	"os"
 	"os/exec"
-	"strconv"
-	"strings"
 
 	"github.com/spf13/cobra"
 )
@@ -101,42 +98,6 @@ func init() {
 	analyzeCmd.Flags().BoolVarP(&autoFix, "fix", "f", false, "Apply auto fix to your issues when available")
 	analyzeCmd.Flags().BoolVar(&doNewPr, "new-pr", false, "Create a new PR on GitHub containing the fixed issues")
 	rootCmd.AddCommand(analyzeCmd)
-	uploadResultsCmd.Flags().StringVarP(&sarifPath, "sarif-path", "s", "", "Path to the SARIF report")
-	uploadResultsCmd.Flags().StringVarP(&commitUuid, "commit-uuid", "c", "", "Commit UUID")
-	uploadResultsCmd.Flags().StringVarP(&projectToken, "project-token", "t", "", "Project token for Codacy API")
-	rootCmd.AddCommand(uploadResultsCmd)
-}
-
-var uploadResultsCmd = &cobra.Command{
-	Use:   "upload",
-	Short: "Uploads a sarif file to Codacy",
-	Long:  "YADA",
-	Run: func(cmd *cobra.Command, args []string) {
-		processSarifAndSendResults(sarifPath, commitUuid, projectToken)
-	},
-}
-
-func processSarifAndSendResults(sarifPath string, commitUuid string, projectToken string) {
-	fmt.Printf("Loading SARIF file from path: %s\n", sarifPath)
-	//Load SARIF file
-	sarifFile, err := os.Open(sarifPath)
-	if err != nil {
-		fmt.Printf("Error opening SARIF file: %v\n", err)
-		panic("panic")
-	}
-	defer sarifFile.Close()
-
-	var sarif Sarif
-	fmt.Println("Parsing SARIF file...")
-	err = json.NewDecoder(sarifFile).Decode(&sarif)
-	if err != nil {
-		fmt.Printf("Error parsing SARIF file: %v\n", err)
-		os.Exit(1)
-	}
-
-	fmt.Println("Loading Codacy patterns...")
-	processSarif(sarif)
-
 }
 
 func loadsToolAndPatterns(toolName string) (Tool, []Pattern) {
@@ -207,115 +168,6 @@ func loadsToolAndPatterns(toolName string) (Tool, []Pattern) {
 	return tool, patterns
 }
 
-func processSarif(sarif Sarif) {
-	var codacyIssues []map[string]interface{}
-
-	for _, run := range sarif.Runs {
-		var toolName = getToolName(strings.ToLower(run.Tool.Driver.Name), run.Tool.Driver.Version)
-		tool, patterns := loadsToolAndPatterns(toolName)
-		for _, result := range run.Results {
-			modifiedType := tool.Prefix + strings.Replace(result.RuleID, "/", "_", -1)
-			pattern := getPatternById(patterns, modifiedType)
-			if pattern == nil {
-				fmt.Printf("Rule '%s' doesn't have a direct mapping on Codacy\n", modifiedType)
-				continue
-			}
-			for _, location := range result.Locations {
-				codacyIssues = append(codacyIssues, map[string]interface{}{
-					"source":   location.PhysicalLocation.ArtifactLocation.URI,
-					"line":     location.PhysicalLocation.Region.StartLine,
-					"type":     modifiedType,
-					"message":  result.Message.Text,
-					"level":    pattern.Level,
-					"category": pattern.Category,
-				})
-			}
-		}
-		groups := make(map[string]map[string]interface{})
-		for _, obj := range codacyIssues {
-			source := obj["source"].(string)
-			if _, ok := groups[source]; !ok {
-				groups[source] = map[string]interface{}{
-					"filename": source,
-					"results":  []interface{}{},
-				}
-			}
-			groups[source]["results"] = append(groups[source]["results"].([]interface{}), map[string]interface{}{
-				"Issue": map[string]interface{}{
-					"patternId": map[string]string{
-						"value": obj["type"].(string),
-					},
-					"filename": source,
-					"message": map[string]string{
-						"text": obj["message"].(string),
-					},
-					"level":    obj["level"].(string),
-					"category": obj["category"].(string),
-					"location": map[string]interface{}{
-						"LineLocation": map[string]int{
-							"line": obj["line"].(int),
-						},
-					},
-				},
-			})
-		}
-		payload := []map[string]interface{}{
-			{
-				"tool": toolName,
-				"issues": map[string]interface{}{
-					"Success": map[string]interface{}{
-						"results": groups,
-					},
-				},
-			},
-		}
-		sendResults(payload, commitUuid, projectToken)
-	}
-
-	resultsFinal(commitUuid, projectToken)
-
-}
-
-func resultsFinal(commitUuid string, projectToken string) {
-	url := fmt.Sprintf("https://api.codacy.com/2.0/commit/%s/resultsFinal", commitUuid)
-	req, _ := http.NewRequest("POST", url, nil)
-	req.Header.Set("Content-Type", "application/json")
-	req.Header.Set("project-token", projectToken)
-
-	client := &http.Client{}
-	resp, err := client.Do(req)
-	if err != nil {
-		fmt.Println("Error:", err)
-		return
-	}
-	defer resp.Body.Close()
-	body, _ := io.ReadAll(resp.Body)
-	fmt.Println("Response:", resp.Status)
-	fmt.Println("Response Body:", string(body))
-}
-
-func getPatternById(patterns []Pattern, patternId string) *Pattern {
-	for _, p := range patterns {
-		if p.ID == patternId {
-			return &p
-		}
-	}
-	return nil
-}
-
-func getMajorVersion(version string) int {
-	parts := strings.Split(version, ".")
-	if len(parts) > 0 {
-		major, err := strconv.Atoi(parts[0])
-		if err != nil {
-			fmt.Println("Error converting major version to integer:", err)
-			return -1
-		}
-		return major
-	}
-	return -1
-}
-
 func getToolName(toolName string, version string) string {
 
 	if toolName == "eslint" {
@@ -334,37 +186,6 @@ func getToolName(toolName string, version string) string {
 	return toolName
 }
 
-func sendResults(payload []map[string]interface{}, commitUuid string, projectToken string) {
-
-	payloadBytes, err := json.Marshal(payload)
-	if err != nil {
-		fmt.Printf("Error marshaling payload: %v\n", err)
-		panic("panic")
-	}
-
-	url := fmt.Sprintf("https://api.codacy.com/2.0/commit/%s/issuesRemoteResults", commitUuid)
-	fmt.Printf("Sending results to URL: %s\n", url)
-	req, err := http.NewRequest("POST", url, bytes.NewBuffer(payloadBytes))
-	if err != nil {
-		fmt.Printf("Error creating request: %v\n", err)
-		os.Exit(1)
-	}
-	req.Header.Set("content-type", "application/json")
-	req.Header.Set("project-token", projectToken)
-
-	resp, err := http.DefaultClient.Do(req)
-	if err != nil {
-		fmt.Printf("Error sending results: %v\n", err)
-		os.Exit(1)
-	}
-	defer resp.Body.Close()
-
-	if resp.StatusCode != http.StatusOK {
-		fmt.Printf("Error sending results, status code: %d\n", resp.StatusCode)
-		os.Exit(1)
-	}
-}
-
 var analyzeCmd = &cobra.Command{
 	Use:   "analyze",
 	Short: "Runs all linters.",