Client Address Bar wont update after 303 redirect response. #2404
Comments
|
Can you show us how you are issuing the redirect and how you are testing it? Because as read this is a description of unexpected behaviour by your browser, not Fresh. |
|
Yes of course. I hope I am not wasting anyone's time by this maybe being a problem with my browser. So I am trying to use a Form for Login like this export const handler: Handlers = {
async GET(req: Request, ctx: FreshContext) {
return await ctx.render();
},
async POST(req: Request, ctx: FreshContext) {
const form = await req.formData();
const email = form.get("email")?.toString();
const password = form.get("password")?.toString();
console.log(`Login for ${email} with ${password}`);
const headers = new Headers();
headers.set("location", "/posts");
return new Response(null, {
status: 303,
headers,
});
},
};
export default function Login() {
return (
<>
<Head>
<title>Login</title>
</Head>
<div className="flex flex-wrap w-full max-w-sm bg-emerald-950 rounded-lg mt-24 p-4 mx-auto">
<h1 className="text-4xl font-extrabold mx-auto">Login</h1>
<div className="py-4 w-full">
<form method="POST">
<label
className="block text-red-100 text-base uppercase font-bold mb-1"
htmlFor="email"
>
EMail
</label>
<input
type="email"
id="email"
name="email"
className="w-full border bg-gray-800/50 rounded px-3 py-2 outline-none text-white"
/>
<label
className="block text-red-100 text-base uppercase font-bold mb-1 mt-4"
htmlFor="password"
>
password
</label>
<input
type="password"
id="password"
name="password"
className="w-full border bg-gray-800/50 rounded px-3 py-2 outline-none text-white"
/>
<div class="flex w-full justify-center pt-4">
<button
type="submit"
className="relative bg-emerald-500 border border-transparent hover:border-emerald-200 py-2 px-6 rounded-md font-bold text-lg text-white"
>
Sign In / Login
</button>
</div>
</form>
</div>
</div>
</>
);
} |
|
Thanks for sharing the reproduction code. That made it a lot easier to understand what your scenario is, what happened and what you did expect to happen. Can confirm that this is a bug. |
|
could it also be that in this case of a redirect, fresh also does not load any islands that are related to that page that the redirect is pointing to. Only those island exist, that already have been loaded. |
|
I am having the same issue. Here's a simpler example to help understand. This code will work and also change the browser URL box correctly when you enter this route into the browser. However if you link to this route from the app by doing: Then this will show the proper page/code from the hello route, but the URL box in browser will say "should_redirect_to_hello" which is wrong. This is avoidable by making sure the app never uses the "should_redirect_to_hello" link, so technically it's not a big issue. But it's still a bug nonetheless. In my opinion, it might be better off left the way it is and developers should never use outdated links in their app unless there's a way to check the correctness of the route without an addition of in-app routing layer which will only bloat the framework. Another option is to use an approach similar to fresh.gen.ts to make sure all links in the app are not broken and not redirects and throw a warning instead. I would like to see the framework keep its simple, non bloated nature. |
The correct Page will load but the Address Bar is still showing the last Page.
At the API the status 303 is used with a new location header.
The text was updated successfully, but these errors were encountered: