-
Notifications
You must be signed in to change notification settings - Fork 21
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
Consider tying enableDebugMode()
availability to third-party cookie eligibility
#57
Comments
Note that this change would not be backwards compatabile in the case where a frame that does not have third-party cookie eligibility runs an Protected Audience auction or Shared Storage operation and tries to use Private Aggregation's debug mode. In that case, we could avoid defining |
Adds logic that allows for debug mode to be disabled even if enableDebugMode() is called. This addresses issue #57. As a drive-by, adds an issue for an assert that is currently hit.
Closing as this has been implemented |
Thanks for the update |
This would ensure parity with what is available through cookies and allow for the temporary debugging mechanism to 'automatically' be removed when third-party cookies are.
The text was updated successfully, but these errors were encountered: