Skip to content

Support diff version and whitespace arguments for diffs batch endpoint

What does this MR do?

It's a next step towards deprecating the regular /diffs endpoint in favor of /diffs_batch.

With this change we can receive pagination arguments at /diffs_batch, even if we fetch diffs from the repository. Though, we won't batch these results yet, which is a work for #32859 (closed). It'll return all content at once, as the regular /diffs.

This mainly avoid using two different endpoints for persisted/batched diffs and repository/non-batch diffs.

#35754 (closed)


Arguments being supported:

  • diff_id
  • start_sha
  • w (include / ignore whitespace)

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability 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
Edited by 🤖 GitLab Bot 🤖

Merge request reports