Commit ee9c8269 authored by Mark Florian's avatar Mark Florian

Fix security report widget for forks

Merge requests from forks run the pipeline in the context of the source
project (i.e., the fork), rather than the target project. This change
makes sure that the correct project is used to get the pipeline jobs
information.

This change also works for same-project merge requests, since in that
case the source project _is_ the target project.
parent 89fb8f24
......@@ -464,7 +464,7 @@ export default {
<security-reports-app
v-if="shouldRenderSecurityReport"
:pipeline-id="mr.pipeline.id"
:project-id="mr.targetProjectId"
:project-id="mr.sourceProjectId"
:security-reports-docs-path="mr.securityReportsDocsPath"
/>
......
---
title: Fix getting security report information on merge requests from forks
merge_request: 49354
author:
type: fixed
......@@ -311,7 +311,7 @@ export default {
<security-reports-app
v-if="shouldRenderBaseSecurityReport"
:pipeline-id="mr.pipeline.id"
:project-id="mr.targetProjectId"
:project-id="mr.sourceProjectId"
:security-reports-docs-path="mr.securityReportsDocsPath"
:sast-comparison-path="mr.sastComparisonPath"
:secret-scanning-comparison-path="mr.secretScanningComparisonPath"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment