fix: added ?return_to for ensureLoggedIn middleware

titanism · titanism · commit d2ec2ba7af65 · 2024-12-19T04:28:44.000-06:00
diff --git a/index.js b/index.js
@@ -189,11 +189,14 @@ class Policies {
         }
       }
 
-      const redirectTo =
+      let redirectTo =
         typeof ctx.state.l === 'function'
           ? ctx.state.l(this.config.loginRoute)
           : this.config.loginRoute;
 
+      if (ctx.url && ctx.url !== '/')
+        redirectTo += `?return_to=${encodeURIComponent(ctx.url)}`;
+
       if (ctx.accepts('html')) ctx.redirect(redirectTo);
       else ctx.body = { message, redirectTo };
 

Original file line number	Diff line number	Diff line change
`@@ -189,11 +189,14 @@ class Policies {`
`189`	`189`	`}`
`190`	`190`	`}`
`191`	`191`
`192`		`- const redirectTo =`
	`192`	`+ let redirectTo =`
`193`	`193`	`typeof ctx.state.l === 'function'`
`194`	`194`	`? ctx.state.l(this.config.loginRoute)`
`195`	`195`	`: this.config.loginRoute;`
`196`	`196`
	`197`	`+ if (ctx.url && ctx.url !== '/')`
	`198`	+ redirectTo += `?return_to=${encodeURIComponent(ctx.url)}`;
	`199`	`+`
`197`	`200`	`if (ctx.accepts('html')) ctx.redirect(redirectTo);`
`198`	`201`	`else ctx.body = { message, redirectTo };`
`199`	`202`