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.

Sign up for GitHub

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

Jump to bottom

Ignore session cookies #2420

Draft

ghostwords wants to merge 1 commit into master from ignore-session-cookies

Member

ghostwords commented Jul 11, 2019 •

edited

Fixes #1545.

Does checking expiration only make sense in onResponseStarted (not onBeforeSendHeaders)? Do we currently ignore some incoming cookies but then record the very same cookies on the way out?
Also ignore short-lived cookies? Ignore only short-lived cookies? Session cookies seem to be much more persistent in practice.
Evaluate impact using badger-sett.
Fix/add new tests


          Ignore session cookies.

fc80b4e

ghostwords force-pushed the master branch from 13bfe52 to 554d04f Compare

July 16, 2020 15:11

ghostwords force-pushed the master branch from ed896f1 to 728cde2 Compare

December 21, 2021 19:44

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment