Skip to content

cleanup: Allow empty requests for RewriteHistory

Will Chandler (ex-GitLab) requested to merge wc/rewrite-history-allow-empty-req into master

With 7630f4b4 (cleanup: Add RewriteHistory RPC, 2024-01-11) we added validation that each request to RewriteHistory contain blobs and or redactions. However, from the client perspective it is simpler to send an initial request with just the repository, then stream in the parameters in subsequent ones.

Update RewriteHistory to allow requests that contain neither blobs or redactions. If no items are passed, then we return success without running git-filter-repo(1).

Related to https://gitlab.com/gitlab-org/gitaly/-/issues/5730

Edited by Will Chandler (ex-GitLab)

Merge request reports