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

feat(ssr)(backport v16): adds server cookie output via response #309

Closed
wants to merge 3 commits into from

Conversation

blakeoxx
Copy link

This fix/feature resolves #266 . It increases the SsrCookieServices ability to use the request and response objects provided by Express to read and write cookies on the server side. This also means that cookies written during SSR will be returned to the client in the response headers.

This PR is intended for a future release of v16 (16.2.0), up to maintainer discretion. If this PR passes code review, I can also port these changes forward to v17.

This PR was originally represented in #304, but was closed to resolve some attribution issues.

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@blakeoxx
Copy link
Author

@pavankjadda this should now be ready for review. Sorry for the wait.

@blakeoxx
Copy link
Author

@pavankjadda anything I need to do for this PR? I see a check failed, but that check was also failing against dependabot's PRs too. So not sure if that's an actual issue.

@blakeoxx blakeoxx changed the title feat(ssr): adds server cookie output via response feat(ssr)(backport v16): adds server cookie output via response May 16, 2024
@pavankjadda pavankjadda deleted the branch stevermeister:v16 May 22, 2024 19:59
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.

2 participants