Skip to content

Set `latest_pipeline_id` and mark project as vulnerable first

What does this MR do?

As StoreReportService can raise an exception, we need to set the latest_pipeline_id and mark the project as vulnerable before we call the StoreReportService. We could also rescue from the exceptions raised by the StoreReportService and raise them again after running the required logic but this way we set the latest_pipeline_id and mark the project as vulnerable as fast as possible.

Related to #271408 (closed).

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Mehmet Emin INAC

Merge request reports

Loading