Simplify Docker curation code

Author: basel1322

commands/curation/curationaudit.go

@@ -137,9 +137,6 @@ func (ca *CurationAuditCommand) checkDockerSupport() (bool, error) {
 	if dockerImageName == "" {
 		return false, nil
 	}
-	if !strings.Contains(dockerImageName, "/") {
-		return false, errorutils.CheckErrorf("invalid docker image format: '%s'. Expected format: 'repo/image:tag' or 'repo/path/image:tag'", dockerImageName)
-	}
 	return true, nil
 }
 

commands/curation/curationaudit_test.go

@@ -1281,36 +1281,6 @@ func Test_getDockerNameScopeAndVersion(t *testing.T) {
 			wantScope:        "",
 			wantVersion:      "latest",
 		},
-		{
-			name:             "Empty id returns empty values",
-			id:               "",
-			artiUrl:          "http://test.jfrog.io/artifactory",
-			repo:             "docker-remote",
-			wantDownloadUrls: nil,
-			wantName:         "",
-			wantScope:        "",
-			wantVersion:      "",
-		},
-		{
-			name:             "Without artiUrl and repo, no download URL",
-			id:               "docker://nginx:1.21.0",
-			artiUrl:          "",
-			repo:             "",
-			wantDownloadUrls: nil,
-			wantName:         "nginx",
-			wantScope:        "",
-			wantVersion:      "1.21.0",
-		},
-		{
-			name:             "Artifactory URL with trailing slash",
-			id:               "docker://nginx:1.21.0",
-			artiUrl:          "http://test.jfrog.io/artifactory/",
-			repo:             "docker-remote",
-			wantDownloadUrls: []string{"http://test.jfrog.io/artifactory/api/docker/docker-remote/v2/nginx/manifests/1.21.0"},
-			wantName:         "nginx",
-			wantScope:        "",
-			wantVersion:      "1.21.0",
-		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {

sca/bom/buildinfo/technologies/docker/docker.go

@@ -30,16 +30,7 @@ var (
 
 func ParseDockerImage(imageName string) (*DockerImageInfo, error) {
 	imageName = strings.TrimSpace(imageName)
-	if imageName == "" {
-		return nil, fmt.Errorf("docker image name is required")
-	}
-	if idx := strings.Index(imageName, ","); idx > 0 {
-		imageName = strings.TrimSpace(imageName[:idx])
-	}
-	imageName = strings.TrimSuffix(imageName, "/")
-
 	info := &DockerImageInfo{Tag: "latest"}
-
 	if idx := strings.LastIndex(imageName, ":"); idx > 0 {
 		afterColon := imageName[idx+1:]
 		if !strings.Contains(afterColon, "/") {
@@ -56,13 +47,6 @@ func ParseDockerImage(imageName string) (*DockerImageInfo, error) {
 	info.Registry = parts[0]
 	info.Repo, info.Image = parseRegistryAndExtract(info.Registry, parts[1:])
 
-	if info.Image == "" {
-		return nil, fmt.Errorf("invalid docker image format: '%s'", imageName)
-	}
-	if info.Repo == "" {
-		return nil, fmt.Errorf("could not determine repository from: '%s'", imageName)
-	}
-
 	log.Debug(fmt.Sprintf("Parsed Docker image - Registry: %s, Repo: %s, Image: %s, Tag: %s",
 		info.Registry, info.Repo, info.Image, info.Tag))
 

sca/bom/buildinfo/technologies/docker/docker_test.go

@@ -109,17 +109,6 @@ func TestParseDockerImage(t *testing.T) {
 			expectedImg:  "nginx",
 			expectedTag:  "latest",
 		},
-		// Error cases
-		{
-			name:        "Empty image name",
-			imageName:   "",
-			expectError: true,
-		},
-		{
-			name:        "No registry",
-			imageName:   "nginx:latest",
-			expectError: true,
-		},
 	}
 
 	for _, tt := range tests {