Title: [258957] trunk/Source/WebCore
Revision
258957
Author
commit-qu...@webkit.org
Date
2020-03-24 16:33:58 -0700 (Tue, 24 Mar 2020)

Log Message

Add console message when legacy TLS is used
https://bugs.webkit.org/show_bug.cgi?id=209444

Patch by Alex Christensen <achristen...@webkit.org> on 2020-03-24
Reviewed by Darin Adler.

* loader/ResourceLoader.cpp:
(WebCore::ResourceLoader::didReceiveResponse):
Indent less, add host to inspector, add release logging without host.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (258956 => 258957)


--- trunk/Source/WebCore/ChangeLog	2020-03-24 23:27:48 UTC (rev 258956)
+++ trunk/Source/WebCore/ChangeLog	2020-03-24 23:33:58 UTC (rev 258957)
@@ -1,3 +1,14 @@
+2020-03-24  Alex Christensen  <achristen...@webkit.org>
+
+        Add console message when legacy TLS is used
+        https://bugs.webkit.org/show_bug.cgi?id=209444
+
+        Reviewed by Darin Adler.
+
+        * loader/ResourceLoader.cpp:
+        (WebCore::ResourceLoader::didReceiveResponse):
+        Indent less, add host to inspector, add release logging without host.
+
 2020-03-24  Antoine Quint  <grao...@apple.com>
 
         [Web Animations] Clean timeline headers up

Modified: trunk/Source/WebCore/loader/ResourceLoader.cpp (258956 => 258957)


--- trunk/Source/WebCore/loader/ResourceLoader.cpp	2020-03-24 23:27:48 UTC (rev 258956)
+++ trunk/Source/WebCore/loader/ResourceLoader.cpp	2020-03-24 23:33:58 UTC (rev 258957)
@@ -501,14 +501,14 @@
     // anything including possibly derefing this; one example of this is Radar 3266216.
     Ref<ResourceLoader> protectedThis(*this);
 
-    if (r.usedLegacyTLS()) {
-        if (m_frame) {
-            if (auto* document = m_frame->document()) {
-                if (!document->usedLegacyTLS()) {
-                    if (auto* page = document->page())
-                        page->console().addMessage(MessageSource::Network, MessageLevel::Warning, "Loaded resource using TLS 1.0 or 1.1, which are deprecated protocols that will be removed. Please use TLS 1.2 or newer instead."_s, 0, document);
-                    document->setUsedLegacyTLS(true);
+    if (r.usedLegacyTLS() && m_frame) {
+        if (auto* document = m_frame->document()) {
+            if (!document->usedLegacyTLS()) {
+                if (auto* page = document->page()) {
+                    RELEASE_LOG_IF_ALLOWED("usedLegacyTLS:");
+                    page->console().addMessage(MessageSource::Network, MessageLevel::Warning, makeString("Loaded resource from ", r.url().host(), " using TLS 1.0 or 1.1, which are deprecated protocols that will be removed. Please use TLS 1.2 or newer instead."), 0, document);
                 }
+                document->setUsedLegacyTLS(true);
             }
         }
     }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to