Make sure that resourceType is a QWebEngineUrlRequestInfo::ResourceTypeImage

249c0301 · Laurent Montel · e0d7ef9b · 249c0301
Commit 249c0301 authored May 29, 2020 by Laurent Montel 😁
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 4 deletions

messageviewer/src/viewer/webengine/cidreferencesurlinterceptor/cidreferencesurlinterceptor.cpp ...dreferencesurlinterceptor/cidreferencesurlinterceptor.cpp +6 -4

No files found.
--- a/messageviewer/src/viewer/webengine/cidreferencesurlinterceptor/cidreferencesurlinterceptor.cpp
+++ b/messageviewer/src/viewer/webengine/cidreferencesurlinterceptor/cidreferencesurlinterceptor.cpp
@@ -37,10 +37,12 @@ bool CidReferencesUrlInterceptor::interceptRequest(QWebEngineUrlRequestInfo &inf
 {
    const QUrl urlRequestUrl(info.requestUrl());
    if (urlRequestUrl.scheme() == QLatin1String("cid")) {
-        const QString newUrl = MessageViewer::WebEngineEmbedPart::self()->contentUrl(
-            urlRequestUrl.path());
-        if (!newUrl.isEmpty()) {
-            info.redirect(QUrl(newUrl));
+        if (info.resourceType() == QWebEngineUrlRequestInfo::ResourceTypeImage) {
+            const QString newUrl = MessageViewer::WebEngineEmbedPart::self()->contentUrl(
+                        urlRequestUrl.path());
+            if (!newUrl.isEmpty()) {
+                info.redirect(QUrl(newUrl));
+            }
        }
    }
    return false;