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

remove unnecessary redirect in server

Browse source
This commit is contained in:
Sean Yesmunt 2020-12-16 16:18:02 -05:00
parent 77739c971a
commit e21b5be6f6
1 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
web/middleware/redirect.js
View file

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