Skip to content

Ignore session cookies#2420

Draft
ghostwords wants to merge 1 commit intomasterfrom
ignore-session-cookies
Draft

Ignore session cookies#2420
ghostwords wants to merge 1 commit intomasterfrom
ignore-session-cookies

Conversation

@ghostwords
Copy link
Copy Markdown
Member

@ghostwords ghostwords commented Jul 11, 2019

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

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.

Include cookie duration in tracking heuristic

1 participant