Escape HTML tags in oss-check

This commit is contained in:
Marcelo Fabri
2017-08-27 07:21:05 -03:00
parent 5a2335033f
commit b635d1ffa8
2 changed files with 6 additions and 3 deletions
+1 -1
View File
@@ -16,7 +16,7 @@ has_app_changes = !modified_files.grep(/Source/).empty?
has_test_changes = !modified_files.grep(/Tests/).empty?
has_danger_changes = !modified_files.grep(/Dangerfile|script\/oss-check|Gemfile/).empty?
has_rules_changes = !modified_files.grep(/Source\/SwiftLintFramework\/Rules/).empty?
has_rules_docs_changes = !modified_files.include?('Rules.md')
has_rules_docs_changes = modified_files.include?('Rules.md')
# Add a CHANGELOG entry for app changes
if !modified_files.include?('CHANGELOG.md') && has_app_changes
+5 -2
View File
@@ -7,6 +7,7 @@
require 'fileutils'
require 'open3'
require 'optparse'
require 'erb'
################################
# Options
@@ -227,10 +228,12 @@ def diff_and_report_changes_to_danger
master = non_empty_lines("#{@working_dir}/master_reports/#{repo.name}.txt")
(master - branch).each do |fixed|
message "This PR fixed a violation in #{repo.name}: [#{fixed}](#{convert_to_link(repo, fixed)})"
escaped_message = ERB::Util.html_escape fixed
message "This PR fixed a violation in #{repo.name}: [#{escaped_message}](#{convert_to_link(repo, fixed)})"
end
(branch - master).each do |violation|
warn "This PR introduced a violation in #{repo.name}: [#{violation}](#{convert_to_link(repo, violation)})"
escaped_message = ERB::Util.html_escape violation
warn "This PR introduced a violation in #{repo.name}: [#{escaped_message}](#{convert_to_link(repo, violation)})"
end
message repo.duration_report
end