Skip to content
Snippets Groups Projects

Add support for batch download operation

Merged Kamil Trzcińśki requested to merge lfs-batch-download into master
  • 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

/cc @marin @jacobvosmaer @yorickpeterse

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
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",
  • This looks good. As discussed offline with @marin and @ayufan, we should probably drop support for GET /info/lfs to avoid future surprising bugs, and only support POST /info/lfs/batch.

  • mentioned in issue #3564 (closed)

  • Added 1 commit:

    • 14d95b05 - Part of tests done [ci skip]
  • Added 1 commit:

    • b8d8292b - Fix upload tests, reformat code and make rubocop happy
  • Marin Jankovski Added 1 commit:

    Added 1 commit:

    • dbc0be1b - Error 501 when client is using deprecated API.
  • Marin Jankovski Title changed from [WIP] Add support for batch download operation to Add support for batch download operation

    Title changed from [WIP] Add support for batch download operation to Add support for batch download operation

  • Marin Jankovski mentioned in merge request !1853 (merged)

    mentioned in merge request !1853 (merged)

  • mentioned in issue #3565 (closed)

  • mentioned in issue #3416 (closed)

  • Marin Jankovski Status changed to merged

    Status changed to merged

  • Marin Jankovski mentioned in commit 807c1a99

    mentioned in commit 807c1a99

  • mentioned in commit b6f0eddc

  • Please register or sign in to reply
    Loading