LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Revert "remove unnecessary redirect in server"

Browse source 
This reverts commit e21b5be6f6.
This commit is contained in:
Sean Yesmunt 2020-12-22 21:26:20 -05:00
parent 75b1bf3a77
commit 63d8f1731e
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
web/middleware/redirect.js
View file

@ -5,7 +5,6 @@ async function redirectMiddleware(ctx, next) {
const requestHost = ctx.host;
const path = ctx.path;
const url = ctx.url;
// Getting err: too many redirects on some urls because of this
// Need a better solution
// const decodedUrl = decodeURIComponent(url);
@ -15,6 +14,11 @@ async function redirectMiddleware(ctx, next) {
// return;
// }
if (path.endsWith('/') && path.length > 1) {
ctx.redirect(url.replace(/\/$/, ''));
return;
}
if (!path.startsWith('/$/') && path.match(/^([^@/:]+)\/([^:/]+)$/)) {
ctx.redirect(url.replace(/^([^@/:]+)\/([^:/]+)(:(\/.*))/, '$1:$2')); // test against path, but use ctx.url to retain parameters
return;