Remove the duplicated const definition

Remove the duplicated const definition for artifact type

Signed-off-by: Wenkai Yin <yinw@vmware.com>
This commit is contained in:
Wenkai Yin 2020-04-10 17:14:11 +08:00
parent 49c9e4f696
commit 7553845b4d
5 changed files with 9 additions and 34 deletions

View File

@ -1,29 +0,0 @@
// Copyright Project Harbor Authors
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package artifact
const (
// ChartType chart type of artifact
ChartType = "CHART"
// CNABType cnab type of artifact
CNABType = "CNAB"
// ImageType image type of artifact
ImageType = "IMAGE"
// UnknownType unknown type of artifact
UnknownType = "UNKNOWN"
)

View File

@ -18,6 +18,7 @@ import (
"context" "context"
"github.com/goharbor/harbor/src/controller/artifact" "github.com/goharbor/harbor/src/controller/artifact"
"github.com/goharbor/harbor/src/controller/artifact/processor/image"
"github.com/goharbor/harbor/src/controller/scanner" "github.com/goharbor/harbor/src/controller/scanner"
models "github.com/goharbor/harbor/src/pkg/scan/dao/scanner" models "github.com/goharbor/harbor/src/pkg/scan/dao/scanner"
) )
@ -88,7 +89,7 @@ func (c *checker) IsScannable(ctx context.Context, art *artifact.Artifact) (bool
func hasCapability(r *models.Registration, a *artifact.Artifact) bool { func hasCapability(r *models.Registration, a *artifact.Artifact) bool {
// use whitelist here because currently only docker image is supported by the scanner // use whitelist here because currently only docker image is supported by the scanner
// https://github.com/goharbor/pluggable-scanner-spec/issues/2 // https://github.com/goharbor/pluggable-scanner-spec/issues/2
whitelist := []string{artifact.ImageType} whitelist := []string{image.ArtifactTypeImage}
for _, t := range whitelist { for _, t := range whitelist {
if a.Type == t { if a.Type == t {
return r.HasCapability(a.ManifestMediaType) return r.HasCapability(a.ManifestMediaType)

View File

@ -23,6 +23,7 @@ import (
"github.com/goharbor/harbor/src/common/models" "github.com/goharbor/harbor/src/common/models"
"github.com/goharbor/harbor/src/common/security" "github.com/goharbor/harbor/src/common/security"
"github.com/goharbor/harbor/src/controller/artifact" "github.com/goharbor/harbor/src/controller/artifact"
"github.com/goharbor/harbor/src/controller/artifact/processor/image"
"github.com/goharbor/harbor/src/controller/project" "github.com/goharbor/harbor/src/controller/project"
"github.com/goharbor/harbor/src/lib" "github.com/goharbor/harbor/src/lib"
securitytesting "github.com/goharbor/harbor/src/testing/common/security" securitytesting "github.com/goharbor/harbor/src/testing/common/security"
@ -59,7 +60,7 @@ func (suite *MiddlewareTestSuite) SetupTest() {
suite.isArtifactSigned = isArtifactSigned suite.isArtifactSigned = isArtifactSigned
suite.artifact = &artifact.Artifact{} suite.artifact = &artifact.Artifact{}
suite.artifact.Type = artifact.ImageType suite.artifact.Type = image.ArtifactTypeImage
suite.artifact.ProjectID = 1 suite.artifact.ProjectID = 1
suite.artifact.RepositoryName = "library/photon" suite.artifact.RepositoryName = "library/photon"
suite.artifact.Digest = "digest" suite.artifact.Digest = "digest"

View File

@ -18,7 +18,8 @@ import (
"fmt" "fmt"
"net/http" "net/http"
"github.com/goharbor/harbor/src/controller/artifact" "github.com/goharbor/harbor/src/controller/artifact/processor/cnab"
"github.com/goharbor/harbor/src/controller/artifact/processor/image"
"github.com/goharbor/harbor/src/controller/project" "github.com/goharbor/harbor/src/controller/project"
"github.com/goharbor/harbor/src/controller/scan" "github.com/goharbor/harbor/src/controller/scan"
"github.com/goharbor/harbor/src/lib" "github.com/goharbor/harbor/src/lib"
@ -114,7 +115,7 @@ func Middleware() func(http.Handler) http.Handler {
if art.IsImageIndex() { if art.IsImageIndex() {
// artifact is image index, skip the checking when it is in the whitelist // artifact is image index, skip the checking when it is in the whitelist
skippingWhitelist := []string{artifact.ImageType, artifact.CNABType} skippingWhitelist := []string{image.ArtifactTypeImage, cnab.ArtifactTypeCNAB}
for _, t := range skippingWhitelist { for _, t := range skippingWhitelist {
if art.Type == t { if art.Type == t {
logger.Debugf("artifact %s@%s is image index and its type is %s in skipping whitelist, "+ logger.Debugf("artifact %s@%s is image index and its type is %s in skipping whitelist, "+

View File

@ -24,6 +24,7 @@ import (
"github.com/goharbor/harbor/src/common/models" "github.com/goharbor/harbor/src/common/models"
"github.com/goharbor/harbor/src/common/security" "github.com/goharbor/harbor/src/common/security"
"github.com/goharbor/harbor/src/controller/artifact" "github.com/goharbor/harbor/src/controller/artifact"
"github.com/goharbor/harbor/src/controller/artifact/processor/image"
"github.com/goharbor/harbor/src/controller/project" "github.com/goharbor/harbor/src/controller/project"
"github.com/goharbor/harbor/src/controller/scan" "github.com/goharbor/harbor/src/controller/scan"
"github.com/goharbor/harbor/src/lib" "github.com/goharbor/harbor/src/lib"
@ -79,7 +80,7 @@ func (suite *MiddlewareTestSuite) SetupTest() {
} }
suite.artifact = &artifact.Artifact{} suite.artifact = &artifact.Artifact{}
suite.artifact.Type = artifact.ImageType suite.artifact.Type = image.ArtifactTypeImage
suite.artifact.ProjectID = 1 suite.artifact.ProjectID = 1
suite.artifact.RepositoryName = "library/photon" suite.artifact.RepositoryName = "library/photon"
suite.artifact.Digest = "digest" suite.artifact.Digest = "digest"