Upstream/harbor
1
0
Fork 0
mirror of https://github.com/goharbor/harbor.git synced 2024-11-26 04:05:40 +01:00
Code Issues Projects Releases Wiki Activity

fix: 404 if report not found when updating report

Browse Source 
Signed-off-by: He Weiwei <hweiwei@vmware.com>
This commit is contained in:
He Weiwei 2021-02-04 07:10:21 +00:00
parent 9cd33ac5e7
commit 060a282e71
1 changed files with 15 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/controller/scan/base_controller.go
View File

@ -659,11 +659,21 @@ func (bc *basicController) UpdateReport(ctx context.Context, report *sca.CheckIn
return errors.Wrap(err, "scan controller: handle job hook") return errors.Wrap(err, "scan controller: handle job hook")
} }
logger := log.G(ctx)
if len(rpl) == 0 { if len(rpl) == 0 {
return errors.New("no report found to update data") fields := log.Fields{
"report_digest": report.Digest,
"registration_uuid": report.RegistrationUUID,
"mime_type": report.MimeType,
}
logger.WithFields(fields).Warningf("no report found to update data")
return errors.NotFoundError(nil).WithMessage("no report found to update data")
} }
log.Infof("Converting report ID %s to the new V2 schema", rpl[0].UUID) logger.Debugf("Converting report ID %s to the new V2 schema", rpl[0].UUID)
_, reportData, err := bc.reportConverter.ToRelationalSchema(ctx, rpl[0].UUID, rpl[0].RegistrationUUID, rpl[0].Digest, report.RawReport) _, reportData, err := bc.reportConverter.ToRelationalSchema(ctx, rpl[0].UUID, rpl[0].RegistrationUUID, rpl[0].Digest, report.RawReport)
if err != nil { if err != nil {
return errors.Wrapf(err, "Failed to convert vulnerability data to new schema for report UUID : %s", rpl[0].UUID) return errors.Wrapf(err, "Failed to convert vulnerability data to new schema for report UUID : %s", rpl[0].UUID)
@ -675,7 +685,9 @@ func (bc *basicController) UpdateReport(ctx context.Context, report *sca.CheckIn
if err := bc.manager.UpdateReportData(ctx, rpl[0].UUID, reportData); err != nil { if err := bc.manager.UpdateReportData(ctx, rpl[0].UUID, reportData); err != nil {
return errors.Wrap(err, "scan controller: handle job hook") return errors.Wrap(err, "scan controller: handle job hook")
} }
log.Infof("Converted report ID %s to the new V2 schema", rpl[0].UUID)
logger.Debugf("Converted report ID %s to the new V2 schema", rpl[0].UUID)
return nil return nil
} }