Skip to content
Snippets Groups Projects

Replace grack with gitlab-git-http-server

Merged Replace grack with gitlab-git-http-server
gitlab-git-http-server into master
Merged username-removed-5302 requested to merge gitlab-git-http-server into master

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
lib/gitlab/backend/grack_auth.rb
132 128
133 129 case git_cmd
134 130 when *Gitlab::GitAccess::DOWNLOAD_COMMANDS
135 if user
131 if !Gitlab.config.gitlab_shell.upload_pack
  • username-removed-5302 Added 27 commits:

    Added 27 commits:

    • 4027a26e...e334ec06 - 26 commits from branch master
    • 90b1ecfa - Merge remote-tracking branch 'gitlab.com/master' into gitlab-git-http-server
  • username-removed-5302 Reassigned to @dzaporozhets

    Reassigned to @dzaporozhets

  • username-removed-5302 mentioned in issue #2158 (closed)

    mentioned in issue #2158 (closed)

  • username-removed-5302 Added 1 commit:

    Added 1 commit:

    • 780bac10 - Be more defensive when running 'kill'
  • username-removed-5302 mentioned in issue #2158 (closed)

    mentioned in issue #2158 (closed)

  • username-removed-444
    username-removed-444 @dzaporozhets ·
    Owner

    @jacobvosmaer I made a brief review. Looks good - I will merge it. Should we add omnibus changes after it?

  • username-removed-444 Status changed to merged

    Status changed to merged

  • username-removed-444 mentioned in commit 2ef16e76

    mentioned in commit 2ef16e76

  • username-removed-5302
    username-removed-5302 @jacobvosmaer ·
    Author Contributor

    Thanks @dzaporozhets . Omnibus changes were already merged, see omnibus-gitlab!450 (merged)

  • username-removed-444
    username-removed-444 @dzaporozhets ·
    Owner

    @jacobvosmaer 👍

    • Please register or sign in to reply