Update Hamlit to 2.6.1
Fixes gitlab-org/gitlab-ce#21025 and gitlab-org/gitlab-ce#21017
This commit is contained in:
parent
83bbca26f4
commit
01fc7633d0
4 changed files with 48 additions and 3 deletions
2
Gemfile
2
Gemfile
|
@ -77,7 +77,7 @@ gem 'rack-cors', '~> 0.4.0', require: 'rack/cors'
|
|||
gem 'kaminari', '~> 0.17.0'
|
||||
|
||||
# HAML
|
||||
gem 'hamlit', '~> 2.5'
|
||||
gem 'hamlit', '~> 2.6.1'
|
||||
|
||||
# Files attachments
|
||||
gem 'carrierwave', '~> 0.10.0'
|
||||
|
|
|
@ -321,7 +321,7 @@ GEM
|
|||
grape-entity (0.4.8)
|
||||
activesupport
|
||||
multi_json (>= 1.3.2)
|
||||
hamlit (2.5.0)
|
||||
hamlit (2.6.1)
|
||||
temple (~> 0.7.6)
|
||||
thor
|
||||
tilt
|
||||
|
@ -863,7 +863,7 @@ DEPENDENCIES
|
|||
gon (~> 6.1.0)
|
||||
grape (~> 0.15.0)
|
||||
grape-entity (~> 0.4.2)
|
||||
hamlit (~> 2.5)
|
||||
hamlit (~> 2.6.1)
|
||||
health_check (~> 2.1.0)
|
||||
hipchat (~> 1.5.0)
|
||||
html-pipeline (~> 1.11.0)
|
||||
|
|
|
@ -97,5 +97,14 @@ describe PageLayoutHelper do
|
|||
expect(tags).to include %q(<meta property="twitter:data1" content="bar" />)
|
||||
end
|
||||
end
|
||||
|
||||
it 'escapes content' do
|
||||
allow(helper).to receive(:page_card_attributes)
|
||||
.and_return(foo: %q{foo" http-equiv="refresh}.html_safe)
|
||||
|
||||
tags = helper.page_card_meta_tags
|
||||
|
||||
expect(tags).to include(%q{content="foo" http-equiv="refresh"})
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
36
spec/views/layouts/_head.html.haml_spec.rb
Normal file
36
spec/views/layouts/_head.html.haml_spec.rb
Normal file
|
@ -0,0 +1,36 @@
|
|||
require 'spec_helper'
|
||||
|
||||
describe 'layouts/_head' do
|
||||
before do
|
||||
stub_template 'layouts/_user_styles.html.haml' => ''
|
||||
end
|
||||
|
||||
it 'escapes HTML-safe strings in page_title' do
|
||||
stub_helper_with_safe_string(:page_title)
|
||||
|
||||
render
|
||||
|
||||
expect(rendered).to match(%{content="foo" http-equiv="refresh"})
|
||||
end
|
||||
|
||||
it 'escapes HTML-safe strings in page_description' do
|
||||
stub_helper_with_safe_string(:page_description)
|
||||
|
||||
render
|
||||
|
||||
expect(rendered).to match(%{content="foo" http-equiv="refresh"})
|
||||
end
|
||||
|
||||
it 'escapes HTML-safe strings in page_image' do
|
||||
stub_helper_with_safe_string(:page_image)
|
||||
|
||||
render
|
||||
|
||||
expect(rendered).to match(%{content="foo" http-equiv="refresh"})
|
||||
end
|
||||
|
||||
def stub_helper_with_safe_string(method)
|
||||
allow_any_instance_of(PageLayoutHelper).to receive(method)
|
||||
.and_return(%q{foo" http-equiv="refresh}.html_safe)
|
||||
end
|
||||
end
|
Loading…
Reference in a new issue