Skip to content
Snippets Groups Projects

Added LFS support to SSH

Merged Patricio Cano requested to merge lfs-authenticate-support into master
All threads resolved!

Required changes to GitLab Shell include the actual handling of the git-lfs-authenticate command and the retrieval of the correct credentials.

Needed for gitlab-org/gitlab-ce!6043

Related to gitlab-org/gitlab-ce#3589

Note: gitlab-org/gitlab-ce!6043 needs to be merged before this one.

cc @jacobvosmaer-gitlab @marin @DouweM

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
  • Patricio Cano Added 1 commit:

    Added 1 commit:

    • 3269940e - Refactored JSON header generation to its own class and added tests for it
  • Patricio Cano Added 1 commit:

    Added 1 commit:

    • 5cb9a05d - Refactored JSON header generation to its own class and added tests for it
  • Patricio Cano Added 1 commit:

    Added 1 commit:

    • 8ed87173 - Added test for old Git LFS clients that submit an extra :oid argument to git-lfs-authenticate
  • Patricio Cano Resolved all discussions

    Resolved all discussions

  • Patricio Cano Added 1 commit:

    Added 1 commit:

    • 2d1fb2a8 - Refactored LFS auth logic to use its own API endpoint.
  • Douwe Maan Marked this merge request as a Work In Progress

    Marked this merge request as a Work In Progress

  • @patricio is @jacobvosmaer-gitlab the person to review this MR again?

  • Douwe Maan
  • Douwe Maan
  • Douwe Maan
  • Douwe Maan
  • Douwe Maan
  • Patricio Cano Added 8 commits:

    Added 8 commits:

    • 2d1fb2a8...c6d8af59 - 3 commits from branch master
    • dbf374e1 - Added LFS support to SSH
    • a64f174d - Refactored JSON header generation to its own class and added tests for it
    • c16f7323 - Added test for old Git LFS clients that submit an extra :oid argument to git-lfs-authenticate
    • f53d09e1 - Refactored LFS auth logic to use its own API endpoint.
    • e6e14fd8 - Style fixes and better tests.
  • Patricio Cano Added 1 commit:

    Added 1 commit:

    • a070ae4a - Style fixes and better tests.
  • Patricio Cano Resolved all discussions

    Resolved all discussions

  • Reassigned to @DouweM

  • Douwe Maan
  • Reassigned to @patricio

  • Patricio Cano Resolved all discussions

    Resolved all discussions

  • Patricio Cano Added 1 commit:

    Added 1 commit:

    • 3c9ef9eb - Properly rescue from JSON parse.
  • Patricio Cano Unmarked this merge request as a Work In Progress

    Unmarked this merge request as a Work In Progress

  • Reassigned to @DouweM

  • Author Contributor

    @DouweM We also need this in order for LFS with SSH to work properly :smiley:

  • Douwe Maan Status changed to merged

    Status changed to merged

  • Douwe Maan Mentioned in commit 6c7f7b4d

    Mentioned in commit 6c7f7b4d

  • @patricio Merged, and tagged v3.6.0! Can you update GITLAB_SHELL_VERSION?

  • Author Contributor

    @DouweM yes, the changelog item for this MR said 3.5.0 and the VERSION file was not updated here. Should I also create an MR here to update those?

  • Please register or sign in to reply
    Loading