-
Notifications
You must be signed in to change notification settings - Fork 2
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
Remove use of cookies #56
Comments
Additional note: we also need to remove the DNT check from Traffic Cop, since this would no longer be required for consent. |
Another vote for option 2: |
Another vote for 2 if/when we need to make changes. |
alexgibson
added a commit
to alexgibson/trafficcop
that referenced
this issue
Jan 12, 2024
alexgibson
added a commit
to alexgibson/trafficcop
that referenced
this issue
Jan 12, 2024
alexgibson
added a commit
to alexgibson/trafficcop
that referenced
this issue
Jan 12, 2024
alexgibson
added a commit
to alexgibson/trafficcop
that referenced
this issue
Jan 12, 2024
alexgibson
added a commit
to alexgibson/trafficcop
that referenced
this issue
Jan 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As part of our upcoming privacy consent work, all non-essential cookies will now require explicit consent to be granted before they can be set. This presents us with the following options for Traffic Cop (TC):
I'm proposing we do 2) because:
The cookies TC does set are certainly useful, but given our constraints I think we're likely to get more overall experiment data if we drop them, so TC can run independently of any consent dialog?
The downside is a visitor might see a different experiment variation if they make a return visit during the experiment window. We would also lose the original referrer in our data during the course of an experiment running.
The text was updated successfully, but these errors were encountered: