Skip to content

Commit 448d5c1

Browse files
committed
Merge pull request #100 from nealharris/neal-gracefully-parse-uris
same_origin? returns false for bad URIs
2 parents b293b54 + 438065a commit 448d5c1

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

lib/secure_headers/headers/content_security_policy.rb

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,8 +177,13 @@ def normalize_reporting_endpoint
177177
def same_origin?
178178
return unless report_uri && request_uri
179179

180-
origin = URI.parse(request_uri)
181-
uri = URI.parse(report_uri)
180+
begin
181+
origin = URI.parse(request_uri)
182+
uri = URI.parse(report_uri)
183+
rescue URI::InvalidURIError
184+
return false
185+
end
186+
182187
uri.host == origin.host && origin.port == uri.port && origin.scheme == uri.scheme
183188
end
184189

0 commit comments

Comments
 (0)