kotovalexarian-likes-gitlab
/
gitlab-org--gitlab-foss
1
0
Fork 0
Code Releases Activity

Don't pass the repository path to Gitlab Shell 

This isn't a requirement, and removes deprecated method calls
This commit is contained in:
Zeger-Jan van de Weg 2018-05-17 15:19:32 +02:00
parent ec7163ae1d
commit a33d591602
No known key found for this signature in database
GPG Key ID: 65F6A8D64A88ABAC
3 changed files with 7 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/api/helpers/internal_helpers.rb
View File

@ -89,12 +89,6 @@ module API
end
end
# Return the repository full path so that gitlab-shell has it when
# handling ssh commands
def repository_path
repository.path_to_repo
end
# Return the Gitaly Address if it is enabled
def gitaly_payload(action)
return unless %w[git-receive-pack git-upload-pack git-upload-archive].include?(action)

2
lib/api/internal.rb
View File

@ -59,7 +59,7 @@ module API
status: true,
gl_repository: gl_repository,
gl_username: user&.username,
repository_path: repository_path,
repository_path: nil,
gitaly: gitaly_payload(params[:action])
}
end

12
spec/requests/api/internal_spec.rb
View File

@ -1,9 +1,9 @@
require 'spec_helper'
describe API::Internal do
let(:user) { create(:user) }
set(:user) { create(:user) }
let(:key) { create(:key, user: user) }
let(:project) { create(:project, :repository, :wiki_repo) }
set(:project) { create(:project, :repository, :wiki_repo) }
let(:secret_token) { Gitlab::Shell.secret_token }
let(:gl_repository) { "project-#{project.id}" }
let(:reference_counter) { double('ReferenceCounter') }
@ -277,7 +277,7 @@ describe API::Internal do
expect(response).to have_gitlab_http_status(200)
expect(json_response["status"]).to be_truthy
expect(json_response["repository_path"]).to eq(project.wiki.repository.path_to_repo)
expect(json_response["repository_path"]).to be_nil
expect(json_response["gl_repository"]).to eq("wiki-#{project.id}")
expect(user).not_to have_an_activity_record
end
@ -289,7 +289,7 @@ describe API::Internal do
expect(response).to have_gitlab_http_status(200)
expect(json_response["status"]).to be_truthy
expect(json_response["repository_path"]).to eq(project.wiki.repository.path_to_repo)
expect(json_response["repository_path"]).to be_nil
expect(json_response["gl_repository"]).to eq("wiki-#{project.id}")
expect(user).to have_an_activity_record
end
@ -301,7 +301,7 @@ describe API::Internal do
expect(response).to have_gitlab_http_status(200)
expect(json_response["status"]).to be_truthy
expect(json_response["repository_path"]).to eq(project.repository.path_to_repo)
expect(json_response["repository_path"]).to be_nil
expect(json_response["gl_repository"]).to eq("project-#{project.id}")
expect(json_response["gitaly"]).not_to be_nil
expect(json_response["gitaly"]["repository"]).not_to be_nil
@ -320,7 +320,7 @@ describe API::Internal do
expect(response).to have_gitlab_http_status(200)
expect(json_response["status"]).to be_truthy
expect(json_response["repository_path"]).to eq(project.repository.path_to_repo)
expect(json_response["repository_path"]).to be_nil
expect(json_response["gl_repository"]).to eq("project-#{project.id}")
expect(json_response["gitaly"]).not_to be_nil
expect(json_response["gitaly"]["repository"]).not_to be_nil