Skip to content

Commit 0886133

Browse files
author
root
committed
Add missing logger prefix
1 parent e65af2e commit 0886133

File tree

2 files changed

+13
-13
lines changed

2 files changed

+13
-13
lines changed

Diff for: lib/libs/git_hosting.rb

+11-11
Original file line numberDiff line numberDiff line change
@@ -602,7 +602,7 @@ def self.fixup_gitolite_admin
602602
unless GitoliteConfig.has_admin_key?
603603
raise GitHostingException, "Cannot repair Admin Key: Key not managed by Redmine!"
604604
end
605-
logger.warn "Attempting to restore repository access key:"
605+
logger.warn "[GitHosting] Attempting to restore repository access key:"
606606

607607
begin
608608
repo_dir = File.join(Dir.tmpdir,"fixrepo",git_user,GitHosting::GitoliteConfig::ADMIN_REPO)
@@ -612,7 +612,7 @@ def self.fixup_gitolite_admin
612612
tmp_conf_dir = File.join(Dir.tmpdir,"fixconf",git_user)
613613
tmp_conf_file = File.join(tmp_conf_dir,gitolite_conf)
614614

615-
logger.warn " Cloning administrative repo directly as #{git_user} in #{repo_dir}"
615+
logger.warn "[GitHosting] Cloning administrative repo directly as #{git_user} in #{repo_dir}"
616616
shell %[rm -rf "#{repo_dir}"] if File.exists?(repo_dir)
617617
admin_repo = "#{GitHostingConf.repository_base}/#{GitHosting::GitoliteConfig::ADMIN_REPO}"
618618
shell %[#{GitHosting.git_user_runner} git clone #{admin_repo} #{repo_dir}]
@@ -638,7 +638,7 @@ def self.fixup_gitolite_admin
638638
# Take basename and remove as many ".pub" as you can
639639
working_basename = /^(.*\/)?([^\/]*?)(\.pub)*$/.match(name)[2]
640640
if first_match || ("#{working_basename}.pub" != File.basename(name))
641-
logger.warn " Removing duplicate administrative key '#{File.basename(name)}' from keydir"
641+
logger.warn "[GitHosting] Removing duplicate administrative key '#{File.basename(name)}' from keydir"
642642
shell %[#{GitHosting.git_user_runner} git --git-dir='#{repo_dir}/.git' --work-tree='#{repo_dir}' rm #{name}]
643643
end
644644
# First name will match this
@@ -656,17 +656,17 @@ def self.fixup_gitolite_admin
656656
# Remove extraneous keys from
657657
extrakeys.each do |keyname|
658658
unless keyname == new_admin_key_name
659-
logger.warn " Removing orphan administrative key '#{keyname}' from gitolite config file"
659+
logger.warn "[GitHosting] Removing orphan administrative key '#{keyname}' from gitolite config file"
660660
conf.delete_admin_keys keyname
661661
end
662662
end
663663

664-
logger.warn " Establishing '#{new_admin_key_name}.pub' as the redmine_git_hosting administrative key"
664+
logger.warn "[GitHosting] Establishing '#{new_admin_key_name}.pub' as the redmine_git_hosting administrative key"
665665

666666
# Add selected key to front of admin list
667667
admin_keys = ([new_admin_key_name] + conf.get_admin_keys).uniq
668668
if (admin_keys.length > 1)
669-
logger.warn " Additional administrative key(s): #{admin_keys[1..-1].map{|x| "'#{x}.pub'"}.join(', ')}"
669+
logger.warn "[GitHosting] Additional administrative key(s): #{admin_keys[1..-1].map{|x| "'#{x}.pub'"}.join(', ')}"
670670
end
671671
conf.set_admin_keys admin_keys
672672
conf.save
@@ -679,19 +679,19 @@ def self.fixup_gitolite_admin
679679
shell %[#{GitHosting.git_user_runner} "git --git-dir='#{repo_dir}/.git' --work-tree='#{repo_dir}' config user.name 'Redmine'"]
680680
shell %[#{GitHosting.git_user_runner} "git --git-dir='#{repo_dir}/.git' --work-tree='#{repo_dir}' commit -m 'Updated by Redmine: Emergency repair of gitolite admin key'"]
681681
begin
682-
logger.warn " Pushing fixes using gl-admin-push"
682+
logger.warn "[GitHosting] Pushing fixes using gl-admin-push"
683683
shell %[#{GitHosting.git_user_runner} "cd #{repo_dir}; gl-admin-push -f"]
684-
logger.warn "Successfully reestablished gitolite admin key!"
684+
logger.warn "[GitHosting] Successfully reestablished gitolite admin key!"
685685
rescue
686-
logger.error "gl-admin-push failed (pre 2.0.3 gitolite?). Trying 'gl-setup #{keydir}/#{new_admin_key_name}.pub'"
686+
logger.error "[GitHosting] gl-admin-push failed (pre 2.0.3 gitolite?). Trying 'gl-setup #{keydir}/#{new_admin_key_name}.pub'"
687687
shell %[#{GitHosting.git_user_runner} "gl-setup #{keydir}/#{new_admin_key_name}.pub"]
688-
logger.warn "Hopefully we have successfully reestablished gitolite admin key."
688+
logger.warn "[GitHosting] Hopefully we have successfully reestablished gitolite admin key."
689689
end
690690
%x[#{GitHosting.git_user_runner} 'rm -rf "#{File.join(Dir.tmpdir,'fixrepo')}"']
691691
%x[rm -rf "#{File.join(Dir.tmpdir,'fixconf')}"]
692692
"Success!"
693693
rescue => e
694-
logger.error "Failed to reestablish gitolite admin key."
694+
logger.error "[GitHosting] Failed to reestablish gitolite admin key."
695695
logger.error e.message
696696
logger.error e.backtrace.join("\n")
697697
%x[#{GitHosting.git_user_runner} 'rm -f ~/id_rsa.pub']

Diff for: lib/libs/gitolite_config.rb

+2-2
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ def save
3939
end
4040
@original_content = content
4141
rescue => e
42-
GitHosting.logger.error "Error trying to write config file: #{e.to_s}"
42+
GitHosting.logger.error "[GitHosting] Error trying to write config file: #{e.to_s}"
4343
end
4444
end
4545

@@ -224,7 +224,7 @@ def load
224224

225225
@original_content = @original_content.join
226226
rescue => e
227-
GitHosting.logger.error "Error trying to read config file: #{e.to_s}"
227+
GitHosting.logger.error "[GitHosting] Error trying to read config file: #{e.to_s}"
228228
end
229229
end
230230

0 commit comments

Comments
 (0)