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 setting cookie max age #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nvdk
Copy link
Member

@nvdk nvdk commented Aug 26, 2024

In some settings 'session cookies' are not preferred, since they can be very long lived. This allows you to set the cookie max age to auto expire cookies

Can be tested using the following docker compose snippet:

  identifier:
    image: nvdk/mu-identifier:feature-cookie-max-age
    links:
      - dispatcher:dispatcher
    environment:
      SESSION_COOKIE_MAX_AGE: 30

This sets a cookie that expires after 30 seconds

In some settings 'session cookies' are not preferred, since they can be very
long lived. This allows set the cookie max age to auto expire cookies
@nvdk nvdk force-pushed the feature/cookie-max-age branch from 01f0fee to 0aea9bf Compare August 26, 2024 10:08
@nvdk nvdk changed the title WIP: allow setting cookie max age allow setting cookie max age Aug 26, 2024
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