removed unmarshalling logic

pkg/detectors/gitlab/v1/gitlab.go

@@ -20,10 +20,6 @@ type Scanner struct {
 	detectors.EndpointSetter
 }
 
-type GitLabMessage struct {
-	Message string `json:"message"`
-}
-
 // Ensure the Scanner satisfies the interfaces at compile time.
 var (
 	_ detectors.Detector           = (*Scanner)(nil)
@@ -111,7 +107,8 @@ func (s Scanner) verifyGitlab(ctx context.Context, resMatch string) (bool, map[s
 		}
 
 		defer res.Body.Close()
-		body, err := io.ReadAll(res.Body)
+
+		bodyBytes, err := io.ReadAll(res.Body)
 		if err != nil {
 			return false, nil, err
 		}
@@ -121,16 +118,14 @@ func (s Scanner) verifyGitlab(ctx context.Context, resMatch string) (bool, map[s
 		// 401 is bad key
 		switch res.StatusCode {
 		case http.StatusOK:
-			return json.Valid(body), nil, nil
+			return json.Valid(bodyBytes), nil, nil
 		case http.StatusForbidden:
 			// check if the user account is blocked or not
-			var apiResp GitLabMessage
-			if err := json.Unmarshal(body, &apiResp); err == nil {
-				if apiResp.Message == BlockedUserMessage {
-					return true, map[string]string{
-						"blocked": "True",
-					}, nil
-				}
+			stringBody := string(bodyBytes)
+			if strings.Contains(stringBody, BlockedUserMessage) {
+				return true, map[string]string{
+					"blocked": "True",
+				}, nil
 			}
 
 			// Good key but not the right scope

pkg/detectors/gitlab/v2/gitlab_v2.go

@@ -2,7 +2,6 @@ package gitlab
 
 import (
 	"context"
-	"encoding/json"
 	"fmt"
 	"io"
 	"net/http"
@@ -95,7 +94,8 @@ func (s Scanner) verifyGitlab(ctx context.Context, resMatch string) (bool, map[s
 			return false, nil, err
 		}
 		defer res.Body.Close()
-		body, err := io.ReadAll(res.Body)
+
+		bodyBytes, err := io.ReadAll(res.Body)
 		if err != nil {
 			return false, nil, err
 		}
@@ -108,13 +108,11 @@ func (s Scanner) verifyGitlab(ctx context.Context, resMatch string) (bool, map[s
 			return true, nil, nil
 		case http.StatusForbidden:
 			// check if the user account is blocked or not
-			var apiResp v1.GitLabMessage
-			if err := json.Unmarshal(body, &apiResp); err == nil {
-				if apiResp.Message == v1.BlockedUserMessage {
-					return true, map[string]string{
-						"blocked": "True",
-					}, nil
-				}
+			stringBody := string(bodyBytes)
+			if strings.Contains(stringBody, v1.BlockedUserMessage) {
+				return true, map[string]string{
+					"blocked": "True",
+				}, nil
 			}
 
 			// Good key but not the right scope