Skip to content

Added example for setting cookie before redirect in middleware#32542

Merged
timneutkens merged 10 commits intovercel:canaryfrom
molebox:middleware-cookie-docs
Dec 21, 2021
Merged

Added example for setting cookie before redirect in middleware#32542
timneutkens merged 10 commits intovercel:canaryfrom
molebox:middleware-cookie-docs

Conversation

@molebox
Copy link
Copy Markdown
Contributor

@molebox molebox commented Dec 15, 2021

This PR adds an example for how to set a cookie before redirecting in middleware

Bug

  • Related issues linked using fixes #number
  • Integration tests added
  • Errors have helpful link attached, see contributing.md

Feature

  • Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR.
  • Related issues linked using fixes: Unable to set cookie before redirect #32424
  • Integration tests added
  • Documentation added
  • Telemetry added. In case of a feature if it's used or not.
  • Errors have helpful link attached, see contributing.md

Documentation / Examples

  • Make sure the linting passes by running yarn lint

Comment thread docs/api-reference/next/server.md
Comment thread docs/api-reference/next/server.md Outdated
Comment thread docs/api-reference/next/server.md Outdated
Comment thread docs/api-reference/next/server.md Outdated
Comment thread docs/api-reference/next/server.md
Comment thread docs/api-reference/next/server.md Outdated
Comment thread docs/api-reference/next/server.md Outdated
Comment thread docs/api-reference/next/server.md
@molebox molebox requested a review from javivelasco December 17, 2021 11:14
javivelasco
javivelasco previously approved these changes Dec 21, 2021
Comment thread docs/api-reference/next/server.md Outdated
Comment thread docs/api-reference/next/server.md Outdated
@timneutkens timneutkens merged commit 4f67edd into vercel:canary Dec 21, 2021
@vercel vercel locked as resolved and limited conversation to collaborators Jan 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants