Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow late redirects in onRender #225

Closed
wants to merge 2 commits into from
Closed

Allow late redirects in onRender #225

wants to merge 2 commits into from

Conversation

almunnings
Copy link

Addressing issue in #144

If there is some late app-based condition met that would require an early 301 redirect, theres no way to do this without generating an error.

This just checks the status code is as expected in the res object before progressing with the send.

Can then override using (for example):

onRender = (res, context) => {
  if (context.myRedirectUrl) {
    res.redirect(302, context.myRedirectUrl)
  }
}

@almunnings almunnings closed this by deleting the head repository Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant