Skip processing for non-GET requests

This commit is contained in:
Stefan Wrobel 2013-08-20 13:55:47 -07:00
parent 81fb8384cb
commit 117f251ff7
3 changed files with 81 additions and 23 deletions

1
.gitignore vendored
View File

@ -3,3 +3,4 @@
Gemfile.lock
pkg/*
*.orig
tmp/

View File

@ -16,7 +16,7 @@ module Rack
end
def skip_processing?
!html? || chunked? || inline? || ignored? || bad_browser?
!html? || chunked? || inline? || ignored? || bad_browser? || !get?
end
def chunked?
@ -38,6 +38,10 @@ module Rack
def html?
@headers['Content-Type'] =~ %r{text/html}
end
def get?
@env['REQUEST_METHOD'] == 'GET'
end
end
end
end

View File

@ -18,32 +18,25 @@ describe Rack::LiveReload::ProcessingSkipAnalyzer do
end
end
describe '#bad_browser?' do
let(:user_agent) { described_class::BAD_USER_AGENTS.first.source }
it { should be_bad_browser }
end
context 'ignored' do
describe '#ignored?' do
let(:options) { { :ignore => [ %r{file} ] } }
context 'not root' do
context 'path contains ignore pattern' do
let(:env) { { 'PATH_INFO' => '/this/file' } }
it { should be_ignored }
end
context 'root' do
context 'root path' do
let(:env) { { 'PATH_INFO' => '/' } }
it { should_not be_ignored }
end
end
context 'not text/html' do
let(:headers) { { 'Content-Type' => 'application/pdf' } }
it { should_not be_html }
describe '#chunked?' do
context 'regular response' do
it { should_not be_chunked }
end
context 'chunked response' do
@ -51,13 +44,15 @@ describe Rack::LiveReload::ProcessingSkipAnalyzer do
it { should be_chunked }
end
end
describe '#inline?' do
context 'inline disposition' do
let(:headers) { { 'Content-Disposition' => 'inline; filename=my_inlined_file' } }
it { should be_inline }
end
end
describe '#ignored?' do
let(:path_info) { 'path info' }
@ -73,5 +68,63 @@ describe Rack::LiveReload::ProcessingSkipAnalyzer do
it { should be_ignored }
end
end
describe '#bad_browser?' do
context 'Firefox' do
it { should_not be_bad_browser }
end
context 'BAD browser' do
let(:user_agent) { described_class::BAD_USER_AGENTS.first.source }
it { should be_bad_browser }
end
end
describe '#html?' do
context 'HTML content' do
let(:headers) { { 'Content-Type' => 'text/html' } }
it { should be_html }
end
context 'PDF content' do
let(:headers) { { 'Content-Type' => 'application/pdf' } }
it { should_not be_html }
end
end
describe '#get?' do
context 'GET request' do
let(:env) { { 'REQUEST_METHOD' => 'GET' } }
it { should be_get }
end
context 'PUT request' do
let(:env) { { 'REQUEST_METHOD' => 'PUT' } }
it { should_not be_get }
end
context 'POST request' do
let(:env) { { 'REQUEST_METHOD' => 'POST' } }
it { should_not be_get }
end
context 'DELETE request' do
let(:env) { { 'REQUEST_METHOD' => 'DELETE' } }
it { should_not be_get }
end
context 'PATCH request' do
let(:env) { { 'REQUEST_METHOD' => 'PATCH' } }
it { should_not be_get }
end
end
end