fix: Avoid redirect loop for Webviews

fix #5007

When the "from" and the "to" location are the same, a redirect should
not be triggered otherwise it will loop and end with a
net::ERR_TOO_MANY_REDIRECTS error

Signed-off-by: Aurélien Pupier <apupier@redhat.com>

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## v1.18.0 - 9/30/2021
 
 - [core, outline-view, file-system, workspace] added breadcrumbs contribution points and renderers to `core` and contributions to other packages. [#9920](https://github.com/eclipse-theia/theia/pull/9920)
+- [plugin] Avoid infinite redirect loop for webview [#5007](https://github.com/eclipse-theia/theia/issues/5007)
 
 <a name="breaking_changes_1.18.0">[Breaking Changes:](#breaking_changes_1.18.0)</a>
 

packages/plugin-ext/src/main/browser/webview/pre/service-worker.js

@@ -248,7 +248,7 @@ async function processLocalhostRequest(event, requestUrl) {
     const origin = requestUrl.origin;
 
     const resolveRedirect = redirectOrigin => {
-        if (!redirectOrigin) {
+        if (!redirectOrigin || requestUrl.origin === redirectOrigin) {
             return fetch(event.request);
         }
         const location = event.request.url.replace(new RegExp(`^${requestUrl.origin}(/|$)`), `${redirectOrigin}$1`);