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

Run without cookies #414

Open
g105b opened this issue Aug 18, 2020 · 2 comments
Open

Run without cookies #414

g105b opened this issue Aug 18, 2020 · 2 comments
Projects

Comments

@g105b
Copy link
Member

g105b commented Aug 18, 2020

When there isn't any need for 'em, don't set 'em.

@g105b g105b added this to Backlog in Overview Oct 9, 2021
@g105b g105b moved this from Backlog to Planned in Overview Oct 16, 2021
@g105b
Copy link
Member Author

g105b commented Oct 17, 2021

This should be possible now the Service Container has moved to lazy initialisation - there doesn't need to be a call to session_start() if there is no use of the session from the page logic. Just needs a test case put in place.

@g105b
Copy link
Member Author

g105b commented Nov 10, 2021

To clarify a common use case, the only time a cookie is required is whenever a POST form is on the page. For typical brochure sites, this is only going to be on the contact page or similar, so the question is, should a cookie only be set on the URL path it's used on, or is this overkill?

@g105b g105b moved this from Planned to Backlog in Overview Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Overview
Backlog
Development

No branches or pull requests

1 participant