[legacy-framework] use decodeURIComponent for router.query.next during page authentication redirect

[remjx]

I ran into an issue where my Page.authenticate = { redirectTo: "/login" } was not working as expected. The initial page was http://localhost:3000/page-i-refreshed and the redirect URL was http://localhost:3000/login?next=%2Fpage-i-refreshed (this is OK so far). But after I logged in, the URL changed to http://localhost:3000/%2Fpage-i-refreshed instead of back to http://localhost:3000/page-i-refreshed which generated a 404.

What are the changes and their implications?

Use decodeURIComponent on the router.query.next path before router.push

Update blitz generate template
Update default login example

Checklist

• Changes covered by tests (tests added if needed)
• PR submitted to blitzjs.com for any user facing changes (N/A)

[remjx]

I'm having trouble getting the tests to run, I made a post in help channel in Discord https://discord.com/channels/802917734999523368/814628067788062750/820540099795091468

[flybayer]

The tests are still failing with a type error. Probably you need to do decodeURIComponent(router.query.next!) or cast it like it was.

Make sure the tests pass. You can run yarn jest from examples/auth/ to run these tests.

[flybayer]

Yay, thank you!!

[blitzjs-bot]

Added @remjx contributions for code