Skip to content

Fix specifying a non-default ref when requesting an archive using the legacy URL

Nick Thomas requested to merge (removed):45507-fix-repository-archive-url into master

What does this MR do?

Fixes the behaviour of URLs like https://gitlab.com/gitlab-org/gitlab-pages/repository/archive.tar.gz?ref=v0.7.0 so they return the requested reference, instead of master.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

This is a regression introduced in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/17225 that subtly breaks a range of automated release processes, leading to people getting the wrong code. At least one downstream project that we know of has been affected (AUR builds of git-latexdiff).

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

/cc @jramsay

Closes #45507 (closed)

Edited by Nick Thomas

Merge request reports