| // Copyright 2016 The Chromium Authors. All rights reserved. |
| // Use of this source code is governed by a BSD-style license that can be |
| // found in the LICENSE file. |
| |
| #include "content/browser/frame_host/ancestor_throttle.h" |
| |
| #include "base/metrics/histogram_macros.h" |
| #include "base/strings/string_split.h" |
| #include "base/strings/string_util.h" |
| #include "base/strings/stringprintf.h" |
| #include "content/browser/frame_host/frame_tree.h" |
| #include "content/browser/frame_host/frame_tree_node.h" |
| #include "content/browser/frame_host/navigation_request.h" |
| #include "content/common/content_security_policy/csp_context.h" |
| #include "content/common/content_security_policy/csp_source_list.h" |
| #include "content/public/browser/browser_thread.h" |
| #include "content/public/browser/navigation_handle.h" |
| #include "content/public/browser/navigation_throttle.h" |
| #include "net/http/http_response_headers.h" |
| #include "services/network/public/cpp/content_security_policy.h" |
| #include "services/network/public/cpp/features.h" |
| #include "third_party/blink/public/mojom/devtools/console_message.mojom.h" |
| #include "url/origin.h" |
| |
| namespace content { |
| |
| namespace { |
| const char kXFrameOptionsSameOriginHistogram[] = "Security.XFrameOptions"; |
| |
| // This enum is used for UMA metrics. Keep these enums up to date with |
| // tools/metrics/histograms/histograms.xml. |
| enum XFrameOptionsHistogram { |
| // A frame is loaded without any X-Frame-Options header. |
| NONE = 0, |
| |
| // X-Frame-Options: DENY. |
| DENY = 1, |
| |
| // X-Frame-Options: SAMEORIGIN. The navigation proceeds and every ancestor |
| // has the same origin. |
| SAMEORIGIN = 2, |
| |
| // X-Frame-Options: SAMEORIGIN. The navigation is blocked because the |
| // top-frame doesn't have the same origin. |
| SAMEORIGIN_BLOCKED = 3, |
| |
| // X-Frame-Options: SAMEORIGIN. The navigation proceeds despite the fact that |
| // there is an ancestor that doesn't have the same origin. |
| SAMEORIGIN_WITH_BAD_ANCESTOR_CHAIN = 4, |
| |
| // X-Frame-Options: ALLOWALL. |
| ALLOWALL = 5, |
| |
| // Invalid 'X-Frame-Options' directive encountered. |
| INVALID = 6, |
| |
| // The frame sets multiple 'X-Frame-Options' header with conflicting values. |
| CONFLICT = 7, |
| |
| // The 'frame-ancestors' CSP directive should take effect instead. |
| BYPASS = 8, |
| |
| // Navigation would have been blocked if we applied 'X-Frame-Options' to |
| // redirects. |
| // |
| // TODO(mkwst): Rename this when we make a decision around |
| // https://crbug.com/835465. |
| REDIRECT_WOULD_BE_BLOCKED = 9, |
| |
| XFRAMEOPTIONS_HISTOGRAM_MAX = REDIRECT_WOULD_BE_BLOCKED |
| }; |
| |
| void RecordXFrameOptionsUsage(XFrameOptionsHistogram usage) { |
| UMA_HISTOGRAM_ENUMERATION( |
| kXFrameOptionsSameOriginHistogram, usage, |
| XFrameOptionsHistogram::XFRAMEOPTIONS_HISTOGRAM_MAX); |
| } |
| |
| bool HeadersContainFrameAncestorsCSP(const net::HttpResponseHeaders* headers) { |
| size_t iter = 0; |
| std::string value; |
| while (headers->EnumerateHeader(&iter, "content-security-policy", &value)) { |
| // A content-security-policy is a semicolon-separated list of directives. |
| for (const auto& directive : base::SplitStringPiece( |
| value, ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_NONEMPTY)) { |
| // The trailing " " is intentional; we'd otherwise match |
| // "frame-ancestors-is-not-this-directive". |
| if (base::StartsWith(directive, "frame-ancestors ", |
| base::CompareCase::INSENSITIVE_ASCII)) |
| return true; |
| } |
| } |
| return false; |
| } |
| |
| } // namespace |
| |
| // static |
| std::unique_ptr<NavigationThrottle> AncestorThrottle::MaybeCreateThrottleFor( |
| NavigationHandle* handle) { |
| DCHECK_CURRENTLY_ON(content::BrowserThread::UI); |
| |
| if (handle->IsInMainFrame()) |
| return nullptr; |
| |
| return std::unique_ptr<NavigationThrottle>(new AncestorThrottle(handle)); |
| } |
| |
| AncestorThrottle::~AncestorThrottle() {} |
| |
| NavigationThrottle::ThrottleCheckResult |
| AncestorThrottle::WillRedirectRequest() { |
| // During a redirect, we don't know which RenderFrameHost we'll end up in, |
| // so we can't log reliably to the console. We should be able to work around |
| // this iff we decide to ship the redirect-blocking behavior, but for now |
| // we'll just skip the console-logging bits to collect metrics. |
| NavigationThrottle::ThrottleCheckResult result = ProcessResponseImpl( |
| LoggingDisposition::DO_NOT_LOG_TO_CONSOLE, false /* is_response_check */); |
| |
| if (result.action() == NavigationThrottle::BLOCK_RESPONSE) |
| RecordXFrameOptionsUsage(XFrameOptionsHistogram::REDIRECT_WOULD_BE_BLOCKED); |
| |
| // TODO(mkwst): We need to decide whether we'll be able to get away with |
| // tightening the XFO check to include redirect responses once we have a |
| // feel for the REDIRECT_WOULD_BE_BLOCKED numbers we're collecting above. |
| // Until then, we'll allow the response to proceed: https://crbug.com/835465. |
| return NavigationThrottle::PROCEED; |
| } |
| |
| NavigationThrottle::ThrottleCheckResult |
| AncestorThrottle::WillProcessResponse() { |
| return ProcessResponseImpl(LoggingDisposition::LOG_TO_CONSOLE, |
| true /* is_response_check */); |
| } |
| |
| NavigationThrottle::ThrottleCheckResult AncestorThrottle::ProcessResponseImpl( |
| LoggingDisposition logging, |
| bool is_response_check) { |
| DCHECK(!navigation_handle()->IsInMainFrame()); |
| |
| NavigationRequest* request = NavigationRequest::From(navigation_handle()); |
| |
| // Downloads should be exempt from checking for X-Frame-Options, so |
| // proceed if this is a download. |
| if (request->IsDownload()) |
| return NavigationThrottle::PROCEED; |
| |
| // Evaluate whether the navigation should be allowed or blocked based on |
| // existing content-security-policy on the response. |
| if (base::FeatureList::IsEnabled( |
| network::features::kOutOfBlinkFrameAncestors)) { |
| if (network::mojom::ContentSecurityPolicyPtr policy = |
| request->response()->head.content_security_policy) { |
| if (auto& frame_ancestors = policy->frame_ancestors) { |
| // TODO(lfg, arthursonzogni): Move the frame-ancestors check to a common |
| // ContentSecurityPolicy object instead of checking directly against the |
| // CSPSourceList. |
| CSPSourceList frame_ancestors_list(*frame_ancestors); |
| frame_ancestors_list.allow_response_redirects = true; |
| FrameTreeNode* parent = request->frame_tree_node()->parent(); |
| bool has_followed_redirect = navigation_handle()->WasServerRedirect(); |
| // Since the navigation hasn't committed yet, we need to create a |
| // CSPContext for the navigation handle. |
| CSPContext csp_context; |
| csp_context.SetSelf(url::Origin::Create(navigation_handle()->GetURL())); |
| while (parent) { |
| if (CSPSourceList::Allow(frame_ancestors_list, |
| parent->current_frame_host() |
| ->GetLastCommittedOrigin() |
| .GetURL(), |
| &csp_context, has_followed_redirect, |
| is_response_check)) { |
| parent = parent->parent(); |
| continue; |
| } |
| auto* frame_to_commit = static_cast<RenderFrameHostImpl*>( |
| navigation_handle()->GetRenderFrameHost()); |
| GURL blocked_url = navigation_handle()->GetURL(); |
| SourceLocation source_location; |
| frame_to_commit->SanitizeDataForUseInCspViolation( |
| has_followed_redirect, CSPDirective::FrameAncestors, &blocked_url, |
| &source_location); |
| std::vector<std::string> report_endpoints; |
| for (auto& url : policy->report_endpoints) |
| report_endpoints.push_back(url.spec()); |
| |
| frame_to_commit->ReportContentSecurityPolicyViolation( |
| // The browser doesn't have the raw CSP text to report in the |
| // message. |
| CSPViolationParams( |
| "frame-ancestors", "frame-ancestors", |
| base::StringPrintf( |
| "Refused to display '%s' in a frame because an ancestor " |
| "violates the frame-ancestors Content Security Policy.", |
| blocked_url.spec().c_str()), |
| blocked_url, report_endpoints, policy->use_reporting_api, |
| "" /* header */, |
| network::mojom::ContentSecurityPolicyType::kEnforce, |
| has_followed_redirect, source_location)); |
| |
| return NavigationThrottle::BLOCK_RESPONSE; |
| } |
| return NavigationThrottle::PROCEED; |
| } |
| } |
| } |
| |
| std::string header_value; |
| HeaderDisposition disposition = |
| ParseHeader(request->GetResponseHeaders(), &header_value); |
| |
| switch (disposition) { |
| case HeaderDisposition::CONFLICT: |
| if (logging == LoggingDisposition::LOG_TO_CONSOLE) |
| ParseError(header_value, disposition); |
| RecordXFrameOptionsUsage(XFrameOptionsHistogram::CONFLICT); |
| return NavigationThrottle::BLOCK_RESPONSE; |
| |
| case HeaderDisposition::INVALID: |
| if (logging == LoggingDisposition::LOG_TO_CONSOLE) |
| ParseError(header_value, disposition); |
| RecordXFrameOptionsUsage(XFrameOptionsHistogram::INVALID); |
| // TODO(mkwst): Consider failing here. |
| return NavigationThrottle::PROCEED; |
| |
| case HeaderDisposition::DENY: |
| if (logging == LoggingDisposition::LOG_TO_CONSOLE) |
| ConsoleError(disposition); |
| RecordXFrameOptionsUsage(XFrameOptionsHistogram::DENY); |
| return NavigationThrottle::BLOCK_RESPONSE; |
| |
| case HeaderDisposition::SAMEORIGIN: { |
| // Block the request when any ancestor is not same-origin. |
| FrameTreeNode* parent = request->frame_tree_node()->parent(); |
| url::Origin current_origin = |
| url::Origin::Create(navigation_handle()->GetURL()); |
| while (parent) { |
| if (!parent->current_origin().IsSameOriginWith(current_origin)) { |
| RecordXFrameOptionsUsage(XFrameOptionsHistogram::SAMEORIGIN_BLOCKED); |
| if (logging == LoggingDisposition::LOG_TO_CONSOLE) |
| ConsoleError(disposition); |
| |
| // TODO(mkwst): Stop recording this metric once we convince other |
| // vendors to follow our lead with XFO: SAMEORIGIN processing. |
| // |
| // https://crbug.com/250309 |
| if (parent->frame_tree()->root()->current_origin().IsSameOriginWith( |
| current_origin)) { |
| RecordXFrameOptionsUsage( |
| XFrameOptionsHistogram::SAMEORIGIN_WITH_BAD_ANCESTOR_CHAIN); |
| } |
| |
| return NavigationThrottle::BLOCK_RESPONSE; |
| } |
| parent = parent->parent(); |
| } |
| RecordXFrameOptionsUsage(XFrameOptionsHistogram::SAMEORIGIN); |
| return NavigationThrottle::PROCEED; |
| } |
| |
| case HeaderDisposition::NONE: |
| RecordXFrameOptionsUsage(XFrameOptionsHistogram::NONE); |
| return NavigationThrottle::PROCEED; |
| case HeaderDisposition::BYPASS: |
| RecordXFrameOptionsUsage(XFrameOptionsHistogram::BYPASS); |
| return NavigationThrottle::PROCEED; |
| case HeaderDisposition::ALLOWALL: |
| RecordXFrameOptionsUsage(XFrameOptionsHistogram::ALLOWALL); |
| return NavigationThrottle::PROCEED; |
| } |
| NOTREACHED(); |
| return NavigationThrottle::BLOCK_RESPONSE; |
| } |
| |
| const char* AncestorThrottle::GetNameForLogging() { |
| return "AncestorThrottle"; |
| } |
| |
| AncestorThrottle::AncestorThrottle(NavigationHandle* handle) |
| : NavigationThrottle(handle) {} |
| |
| void AncestorThrottle::ParseError(const std::string& value, |
| HeaderDisposition disposition) { |
| DCHECK(disposition == HeaderDisposition::CONFLICT || |
| disposition == HeaderDisposition::INVALID); |
| if (!navigation_handle()->GetRenderFrameHost()) |
| return; // Some responses won't have a RFH (i.e. 204/205s or downloads). |
| |
| std::string message; |
| if (disposition == HeaderDisposition::CONFLICT) { |
| message = base::StringPrintf( |
| "Refused to display '%s' in a frame because it set multiple " |
| "'X-Frame-Options' headers with conflicting values " |
| "('%s'). Falling back to 'deny'.", |
| navigation_handle()->GetURL().spec().c_str(), value.c_str()); |
| } else { |
| message = base::StringPrintf( |
| "Invalid 'X-Frame-Options' header encountered when loading '%s': " |
| "'%s' is not a recognized directive. The header will be ignored.", |
| navigation_handle()->GetURL().spec().c_str(), value.c_str()); |
| } |
| |
| // Log a console error in the parent of the current RenderFrameHost (as |
| // the current RenderFrameHost itself doesn't yet have a document). |
| navigation_handle()->GetRenderFrameHost()->GetParent()->AddMessageToConsole( |
| blink::mojom::ConsoleMessageLevel::kError, message); |
| } |
| |
| void AncestorThrottle::ConsoleError(HeaderDisposition disposition) { |
| DCHECK(disposition == HeaderDisposition::DENY || |
| disposition == HeaderDisposition::SAMEORIGIN); |
| if (!navigation_handle()->GetRenderFrameHost()) |
| return; // Some responses won't have a RFH (i.e. 204/205s or downloads). |
| |
| std::string message = base::StringPrintf( |
| "Refused to display '%s' in a frame because it set 'X-Frame-Options' " |
| "to '%s'.", |
| navigation_handle()->GetURL().spec().c_str(), |
| disposition == HeaderDisposition::DENY ? "deny" : "sameorigin"); |
| |
| // Log a console error in the parent of the current RenderFrameHost (as |
| // the current RenderFrameHost itself doesn't yet have a document). |
| navigation_handle()->GetRenderFrameHost()->GetParent()->AddMessageToConsole( |
| blink::mojom::ConsoleMessageLevel::kError, message); |
| } |
| |
| AncestorThrottle::HeaderDisposition AncestorThrottle::ParseHeader( |
| const net::HttpResponseHeaders* headers, |
| std::string* header_value) { |
| DCHECK(header_value); |
| if (!headers) |
| return HeaderDisposition::NONE; |
| |
| // Process the 'X-Frame-Options header as per Section 2 of RFC7034: |
| // https://tools.ietf.org/html/rfc7034#section-2 |
| // |
| // Note that we do not support the 'ALLOW-FROM' value, and we special-case |
| // the invalid "ALLOWALL" value due to its prevalance in the wild. |
| HeaderDisposition result = HeaderDisposition::NONE; |
| size_t iter = 0; |
| std::string value; |
| while (headers->EnumerateHeader(&iter, "x-frame-options", &value)) { |
| HeaderDisposition current = HeaderDisposition::INVALID; |
| |
| base::StringPiece trimmed = |
| base::TrimWhitespaceASCII(value, base::TRIM_ALL); |
| if (!header_value->empty()) |
| header_value->append(", "); |
| header_value->append(trimmed.as_string()); |
| |
| if (base::LowerCaseEqualsASCII(trimmed, "deny")) |
| current = HeaderDisposition::DENY; |
| else if (base::LowerCaseEqualsASCII(trimmed, "allowall")) |
| current = HeaderDisposition::ALLOWALL; |
| else if (base::LowerCaseEqualsASCII(trimmed, "sameorigin")) |
| current = HeaderDisposition::SAMEORIGIN; |
| else |
| current = HeaderDisposition::INVALID; |
| |
| if (result == HeaderDisposition::NONE) |
| result = current; |
| else if (result != current) |
| result = HeaderDisposition::CONFLICT; |
| } |
| |
| // If 'X-Frame-Options' would potentially block the response, check whether |
| // the 'frame-ancestors' CSP directive should take effect instead. See |
| // https://www.w3.org/TR/CSP/#frame-ancestors-and-frame-options |
| if (result != HeaderDisposition::NONE && |
| result != HeaderDisposition::ALLOWALL && |
| HeadersContainFrameAncestorsCSP(headers)) { |
| DCHECK(!base::FeatureList::IsEnabled( |
| network::features::kOutOfBlinkFrameAncestors)); |
| // TODO(mkwst): 'frame-ancestors' is currently handled in Blink. We should |
| // handle it here instead. Until then, don't block the request, and let |
| // Blink handle it. https://crbug.com/555418 |
| return HeaderDisposition::BYPASS; |
| } |
| return result; |
| } |
| |
| } // namespace content |