Merge branch 'author_attributes_clean' into 'master'

Author attributes clean

See merge request !1254

1 files changed, 6 insertions, 2 deletions

diff --git a/app/helpers/commits_helper.rb b/app/helpers/commits_helper.rb
index 0e0532b65b2..36adeadd8a5 100644
--- a/app/helpers/commits_helper.rb
+++ b/app/helpers/commits_helper.rb
@@ -87,8 +87,8 @@ module CommitsHelper
   #  avatar: true will prepend the avatar image
   #  size:   size of the avatar image in px
   def commit_person_link(commit, options = {})
-    source_name = commit.send "#{options[:source]}_name".to_sym
-    source_email = commit.send "#{options[:source]}_email".to_sym
+    source_name = clean(commit.send "#{options[:source]}_name".to_sym)
+    source_email = clean(commit.send "#{options[:source]}_email".to_sym)
 
     user = User.find_for_commit(source_email, source_name)
     person_name = user.nil? ? source_name : user.name
@@ -124,4 +124,8 @@ module CommitsHelper
   def truncate_sha(sha)
     Commit.truncate_sha(sha)
   end
+
+  def clean(string)
+    Sanitize.clean(string, remove_contents: true)
+  end
 end