Add support for batch download operation
- Drops Accept for all download requests,
- Allows to do batch download for public projects and non-authorized users
- Returns 501 for legacy API with message to upgrade the client
Merge request reports
Activity
Reassigned to @marin
Current implementation will work if we disable
lfs batch
(the new API 1.0):git config lfs.batch false
As for downloading.
Edited by Kamil TrzcińśkiIt appears that we implement part of the API 0.5 and part of API 1.0:
- we support downloading only using old API 0.5
- we support uploading only using a new Batch API
279 312 { 280 313 'href' => "#{@origin_project.http_url_to_repo}/gitlab-lfs/objects/#{oid}", 281 314 'header' => { 282 'Accept' => "application/vnd.git-lfs+json; charset=utf-8", Why did you remove this header? According to spec it requires the header. At least for the Content-type.
Edited by Marin Jankovski
mentioned in issue #3564 (closed)
This is broken since release of git-lfs 1.0.1: https://github.com/github/git-lfs/commit/f56cd54ad7f8e13df6415a568f5681394bd44b31
Added 1 commit:
- 14d95b05 - Part of tests done [ci skip]
Added 1 commit:
- b8d8292b - Fix upload tests, reformat code and make rubocop happy
Added 1 commit:
- dbc0be1b - Error 501 when client is using deprecated API.
mentioned in merge request !1853 (merged)
mentioned in issue #3565 (closed)
mentioned in issue #3416 (closed)
mentioned in commit 807c1a99
mentioned in commit b6f0eddc