Remove redundant decodeURIComponent on returnUrl

tobihagemann · tobihagemann · commit cca054659ae4 · 2026-03-06T13:49:53.000+01:00
diff --git a/assets/js/hubsubscription.js b/assets/js/hubsubscription.js
@@ -24,9 +24,9 @@ class HubSubscription {
       }
     }
     this._subscriptionData.hubId = this._subscriptionData.hubId ?? searchParams.get('hub_id');
-    let encodedReturnUrl = fragmentParams.get('returnUrl') ?? searchParams.get('return_url');
-    if (encodedReturnUrl) {
-      this._subscriptionData.returnUrl = decodeURIComponent(encodedReturnUrl);
+    let returnUrl = fragmentParams.get('returnUrl') ?? searchParams.get('return_url');
+    if (returnUrl) {
+      this._subscriptionData.returnUrl = returnUrl;
     }
     this._subscriptionData.session = searchParams.get('session');
     if (this._subscriptionData.hubId && this._subscriptionData.hubId.length > 0 && this._subscriptionData.returnUrl && this._subscriptionData.returnUrl.length > 0) {

Original file line number	Diff line number	Diff line change
`@@ -24,9 +24,9 @@ class HubSubscription {`
`24`	`24`	`}`
`25`	`25`	`}`
`26`	`26`	`this._subscriptionData.hubId = this._subscriptionData.hubId ?? searchParams.get('hub_id');`
`27`		`- let encodedReturnUrl = fragmentParams.get('returnUrl') ?? searchParams.get('return_url');`
`28`		`- if (encodedReturnUrl) {`
`29`		`- this._subscriptionData.returnUrl = decodeURIComponent(encodedReturnUrl);`
	`27`	`+ let returnUrl = fragmentParams.get('returnUrl') ?? searchParams.get('return_url');`
	`28`	`+ if (returnUrl) {`
	`29`	`+ this._subscriptionData.returnUrl = returnUrl;`
`30`	`30`	`}`
`31`	`31`	`this._subscriptionData.session = searchParams.get('session');`
`32`	`32`	`if (this._subscriptionData.hubId && this._subscriptionData.hubId.length > 0 && this._subscriptionData.returnUrl && this._subscriptionData.returnUrl.length > 0) {`