Skip to content

fix(auth0-server-js): StatelessStateStore cookie options - #32

Merged
frederikprijck merged 2 commits into
auth0:mainfrom
CarsonF:bugfix/cookie-options-input-type
Jun 18, 2025
Merged

fix(auth0-server-js): StatelessStateStore cookie options#32
frederikprijck merged 2 commits into
auth0:mainfrom
CarsonF:bugfix/cookie-options-input-type

Conversation

@CarsonF

@CarsonF CarsonF commented Jun 18, 2025

Copy link
Copy Markdown
Contributor

Description

Fixes constructor param type to allow age configuration, as the runtime logic already considers.
Allows sameSite & secure to be configured as well.
These changes just mirror the Stateful implementation.

new StatelessStateStore({
  absoluteDuration: '..',
  cookie: { sameSite: 'strict' }
})

Testing

Describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.

Also include details of the environment this PR was developed in (language/platform/browser version).

  • This change adds test coverage for new/changed/fixed functionality

Checklist

  • I have added documentation for new/changed functionality in this PR or in auth0.com/docs
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used, if not the default branch

@frederikprijck frederikprijck left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I missed this!

@frederikprijck
frederikprijck merged commit eaeb16b into auth0:main Jun 18, 2025
3 checks passed
@CarsonF
CarsonF deleted the bugfix/cookie-options-input-type branch June 18, 2025 15:30
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