Skip to content

WIP: Resolve "View arbitrary commit in merge request for review purposes"

What does this MR do?

Added new classes MergeRequestContextCommit and MergeRequestContextCommitDiffFile.

Added 3 new routes:

1- post :add_context_commits

2- post :remove_context_commits

3- get :context_diff

Screenshots

N/A

Does this MR meet the acceptance criteria?

Conformity

Performance and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • 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

Closes #62817 (moved)

Edited by Paulo Vitor Magacho

Merge request reports