Commit ff33e05b authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Merge pull request #81 from corny/patch-1

avoid @user_tried
parents 3a16acf3 4853c677
...@@ -12,7 +12,6 @@ class GitlabShell ...@@ -12,7 +12,6 @@ class GitlabShell
@origin_cmd = ENV['SSH_ORIGINAL_COMMAND'] @origin_cmd = ENV['SSH_ORIGINAL_COMMAND']
@config = GitlabConfig.new @config = GitlabConfig.new
@repos_path = @config.repos_path @repos_path = @config.repos_path
@user_tried = false
end end
def exec def exec
...@@ -75,10 +74,9 @@ class GitlabShell ...@@ -75,10 +74,9 @@ class GitlabShell
def user def user
# Can't use "@user ||=" because that will keep hitting the API when @user is really nil! # Can't use "@user ||=" because that will keep hitting the API when @user is really nil!
if @user_tried if instance_variable_defined?('@user')
@user @user
else else
@user_tried = true
@user = api.discover(@key_id) @user = api.discover(@key_id)
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment