[Bug?]: throwing redirect in route load function is not handled

[maksimsemenov]

[Bug?]: throwing redirect in route load function is not handled

[Brendonovich]

Throwing the redirect also works, but the important part is calling loadData outside of the load function, since thrown/returned redirects will be ignored when called from load.
https://stackblitz.com/edit/github-fk8exj-mcht9p?file=src%2Froutes%2Findex.tsx

[brenelz]

I feel this is intended behavior at the moment. Thinking we can probably close this issue?

[brenelz]

So a couple things you need to do. You need to return instead of throw the redirect. Secondly you need to wrap in the cache function. Also you need to call the function in createAsync()

const loadData = cache(() => {
  'use server'
  return redirect('/about')
}, "load-data");

export const route = {
  load: () => loadData()
}

export default function Home() {
  const data = createAsync(() => loadData());
}

[maksimsemenov]

What if I don't want to cache the request? For example, if I want to validate some auth-link with some hash parameter, and then want to redirect user. I don't think I want to cache the response in this case.

[Brendonovich]

Use action, it's for pretty much everything that isn't a data fetch

[maksimsemenov]

Can I use action on page load? The use-case I'm trying to solve is:

1. I send email to the user with a unique link
2. User clicks the link and is taken to the link validation page
3. I validate link on the server
4. If validation is successful, I redirect the user
5. If validation is not successful, I want to show some error state with some next action, so user is not lost.

I also, don't want to cash that response, so if link is clicked twice, it is no longer valid

[Brendonovich]

Yeah that should work fine

[maksimsemenov]

So should it be like this:

const loadData = action(() => {
  'use server'
  return redirect('/about')
});

export const route = {
  load: () => loadData()
}

export default function Home() {
  const data = createAsync(() => loadData());
}

[ryansolid]

Yeah this is by design since the load function is more of like a lifecycle hook and the server function is just an async function. One could catch the errored response themselves and then call the client side navigation APIs with it but I don't know if that is what you are going for. The name cache is probably misleading because on the server it only lasts the lifetime of the request. It is possible it would just work. We're reviewing some of the naming.

I think an action is fine, but I think cache more fits the semantics because it is getting is validated on a hash. But either probably is fine here.