Also ensure to not process empty Location headers
This commit is contained in:
@@ -22,7 +22,7 @@ class WpUser < WpItem
|
|||||||
if response.code == 301 # login in location?
|
if response.code == 301 # login in location?
|
||||||
location = response.headers_hash['Location']
|
location = response.headers_hash['Location']
|
||||||
|
|
||||||
return unless location
|
return if location.nil? || location.empty?
|
||||||
|
|
||||||
@login = Existable.login_from_author_pattern(location)
|
@login = Existable.login_from_author_pattern(location)
|
||||||
@display_name = Existable.display_name_from_body(
|
@display_name = Existable.display_name_from_body(
|
||||||
|
|||||||
@@ -133,6 +133,15 @@ shared_examples 'WpUser::Existable' do
|
|||||||
@display_name = nil
|
@display_name = nil
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
context 'when the location is empty' do
|
||||||
|
let(:location) { '' }
|
||||||
|
|
||||||
|
it 'returns nil' do
|
||||||
|
@login = nil
|
||||||
|
@display_name = nil
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
context 'with a 200' do
|
context 'with a 200' do
|
||||||
|
|||||||
Reference in New Issue
Block a user