feat: add support for automatic session driver config #13145
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🦋 Changeset detectedLatest commit: f673dee The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
github-actions
bot
added
pkg: astro
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
CodSpeed Performance ReportMerging #13145 will not alter performanceComparing Summary
|
github-actions
bot
added
the
pkg: integration
|
!preview adapter-sessions |
|
!preview adapter-sessions |
|
❌ Snapshot release failed. |
|
!preview adapter-sessions |
|
Snapshots have been released for the following packages:
Build LogPublish Log |
ascorbic
force-pushed
the
session-config2
branch
from
8a63aac to
db142f0
Compare
|
|
|
@ematipico I will actually bump it to 5.3.0, because this PR changes the API |
ematipico
reviewed
Feb 10, 2025
| return driver; | ||
| } | ||
|
|
||
| export function validateSessionConfig(settings: AstroSettings): void { |
There was a problem hiding this comment.
Why don't we validate the configuration inside the zod schema using refine or superRefine?
EDIT: I understand why we need a function, I think it's worth adding a comment that explains that
sarah11918
reviewed
Feb 11, 2025
Co-authored-by: Sarah Rainsberger <[email protected]>
ematipico
approved these changes
Feb 12, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
Automatically configures session storage with the Node and Netlify adapters. If the
experimental.sessionflag is enabled, the Node adapter will configurefssession storage and Netlify will configurenetlify-blobssession storage.This also changes the
experimental.sessionoption to a boolean flag and moves session config to a top-level value. This change is to allow the new automatic session driver support. You now need to separately enable theexperimental.sessionflag, and then configure the session driver using the top-levelsessionkey if providing manual configuration.defineConfig({ // ... experimental: { - session: { - driver: 'upstash', - }, + session: true, }, + session: { + driver: 'upstash', + }, });You no longer need to configure a session driver if you are using an adapter that supports automatic session driver configuration and wish to use its default settings.
defineConfig({ adapter: node({ mode: "standalone", }), experimental: { - session: { - driver: 'fs', - cookie: 'astro-cookie', - }, + session: true, }, + session: { + cookie: 'astro-cookie', + }, });However, you can still manually configure additional driver options or choose a non-default driver to use with your adapter with the new top-level
sessionconfig option. For more information, see the experimental session docs.Testing
Added test cases for the Node and Netlify adapters. The Netlify tests use the local mock blobs server.
Docs
Updated the docs for the config options. This adds several new error types, so I have moved session errors into their own section. I will do a separate PR to the docs repo to update the experimental feature page.