Skip to content

Commit a1c4bd7

Browse files
authored
fix(server): secrets inspectation for the config analyzer in client server mode (#8418)
1 parent 613fc71 commit a1c4bd7

File tree

4 files changed

+222
-190
lines changed

4 files changed

+222
-190
lines changed

pkg/rpc/convert.go

Lines changed: 27 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -146,10 +146,7 @@ func ConvertToRPCCode(code ftypes.Code) *common.Code {
146146
func ConvertToRPCSecrets(secrets []ftypes.Secret) []*common.Secret {
147147
var rpcSecrets []*common.Secret
148148
for _, s := range secrets {
149-
rpcSecrets = append(rpcSecrets, &common.Secret{
150-
Filepath: s.FilePath,
151-
Findings: ConvertToRPCSecretFindings(s.Findings),
152-
})
149+
rpcSecrets = append(rpcSecrets, ConvertToRPCSecret(&s))
153150
}
154151
return rpcSecrets
155152
}
@@ -525,11 +522,8 @@ func ConvertFromRPCSecretFindings(rpcFindings []*common.SecretFinding) []ftypes.
525522

526523
func ConvertFromRPCSecrets(recSecrets []*common.Secret) []ftypes.Secret {
527524
var secrets []ftypes.Secret
528-
for _, secret := range recSecrets {
529-
secrets = append(secrets, ftypes.Secret{
530-
FilePath: secret.Filepath,
531-
Findings: ConvertFromRPCSecretFindings(secret.Findings),
532-
})
525+
for _, recSecret := range recSecrets {
526+
secrets = append(secrets, *ConvertFromRPCSecret(recSecret))
533527
}
534528
return secrets
535529
}
@@ -794,6 +788,7 @@ func ConvertFromRPCPutArtifactRequest(req *cache.PutArtifactRequest) ftypes.Arti
794788
DockerVersion: req.ArtifactInfo.DockerVersion,
795789
OS: req.ArtifactInfo.Os,
796790
HistoryPackages: ConvertFromRPCPkgs(req.ArtifactInfo.HistoryPackages),
791+
Secret: ConvertFromRPCSecret(req.ArtifactInfo.Secret),
797792
}
798793
}
799794

@@ -854,6 +849,7 @@ func ConvertToRPCArtifactInfo(imageID string, imageInfo ftypes.ArtifactInfo) *ca
854849
DockerVersion: imageInfo.DockerVersion,
855850
Os: imageInfo.OS,
856851
HistoryPackages: ConvertToRPCPkgs(imageInfo.HistoryPackages),
852+
Secret: ConvertToRPCSecret(imageInfo.Secret),
857853
},
858854
}
859855
}
@@ -1015,3 +1011,25 @@ func ConvertFromDeleteBlobsRequest(deleteBlobsRequest *cache.DeleteBlobsRequest)
10151011
}
10161012
return deleteBlobsRequest.GetBlobIds()
10171013
}
1014+
1015+
// ConvertFromRPCSecret converts common.Secret to fanal.Secret
1016+
func ConvertFromRPCSecret(rpcSecret *common.Secret) *ftypes.Secret {
1017+
if rpcSecret == nil {
1018+
return nil
1019+
}
1020+
return &ftypes.Secret{
1021+
FilePath: rpcSecret.Filepath,
1022+
Findings: ConvertFromRPCSecretFindings(rpcSecret.Findings),
1023+
}
1024+
}
1025+
1026+
// ConvertToRPCSecret converts fanal.Secret to common.Secret
1027+
func ConvertToRPCSecret(secret *ftypes.Secret) *common.Secret {
1028+
if secret == nil {
1029+
return nil
1030+
}
1031+
return &common.Secret{
1032+
Filepath: secret.FilePath,
1033+
Findings: ConvertToRPCSecretFindings(secret.Findings),
1034+
}
1035+
}

0 commit comments

Comments
 (0)