refactor project to include some nicer EE stuff
This commit is contained in:
parent
f26389a02a
commit
341c1181d9
1 changed files with 3 additions and 8 deletions
|
@ -204,7 +204,7 @@ class Project < ActiveRecord::Base
|
|||
state :finished
|
||||
state :failed
|
||||
|
||||
after_transition any => :finished, do: :clear_import_data
|
||||
after_transition any => :finished, do: :reset_cache_and_import_attrs
|
||||
end
|
||||
|
||||
class << self
|
||||
|
@ -360,7 +360,7 @@ class Project < ActiveRecord::Base
|
|||
end
|
||||
end
|
||||
|
||||
def clear_import_data
|
||||
def reset_cache_and_import_attrs
|
||||
update(import_error: nil)
|
||||
|
||||
ProjectCacheWorker.perform_async(self.id)
|
||||
|
@ -426,12 +426,7 @@ class Project < ActiveRecord::Base
|
|||
end
|
||||
|
||||
def safe_import_url
|
||||
result = URI.parse(self.import_url)
|
||||
result.password = '*****' unless result.password.nil?
|
||||
result.user = '*****' unless result.user.nil? || result.user == "git" #tokens or other data may be saved as user
|
||||
result.to_s
|
||||
rescue
|
||||
self.import_url
|
||||
Gitlab::UrlSanitizer.new(import_url).masked_url
|
||||
end
|
||||
|
||||
def check_limit
|
||||
|
|
Loading…
Reference in a new issue