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

Suppress "unable to send message to tab" for protected pages #104

Open
rhelmer opened this issue May 7, 2021 · 2 comments
Open

Suppress "unable to send message to tab" for protected pages #104

rhelmer opened this issue May 7, 2021 · 2 comments
Assignees

Comments

@rhelmer
Copy link
Contributor

rhelmer commented May 7, 2021

I've filed this issue upstream, but since RS01 shipped with a fork of this module I wanted to track it here too:
mozilla-rally/web-science#79

Since this was noticed by an internal user (@cpeterso, reported in Slack) we should assume that other users might be annoyed by this. I'd like to pick it up in an update.

@rhelmer rhelmer self-assigned this May 7, 2021
@rhelmer
Copy link
Contributor Author

rhelmer commented May 8, 2021

From the upstream bug:

DataCollection.sendPing - options: {"studyName":"rally-zero-one","addPioneerId":true,"overridePioneerId":"f6b4fbd8-9caf-422a-b79a-35115647da08","encryptionKeyId":"zero-one","publicKey":{"crv":"P-256","kid":"zero-one","kty":"EC","x":"edhPpqhgK9dD7NaqhQ7Ckw9sU6b39X7XB8HnA366Rjs","y":"GzsfM19n-iH-DVR0iKEoA8BE2CFF46wR__siJ3SdiNs"},"schemaName":"measurements","schemaVersion":1,"schemaNamespace":"rally-zero-one"} payload: {"pageId":"e22a87d715d02e4f47422d1cf43d4797","origin":"https://www.cnn.com","referrerOrigin":"","pageVisitStartTime":1620417062877,"pageVisitStopTime":1620417067890,"duration":403,"maxRelativeScrollDepth":0,"maxPixelScrollDepth":0,"scrollHeight":0,"eventTerminationReason":"window-focus-lost","title":"CNN - Breaking News, Latest News and Videos","ogType":"website","description":"View the latest news and breaking news today for U.S., world, weather, entertainment, politics and health at CNN.com.","eventStartTime":1620417067487,"eventStopTime":1620417067890,"eventType":"attention"}

@hamilton we should suppress this entirely from non-debug builds ^

@rhelmer
Copy link
Contributor Author

rhelmer commented May 8, 2021

@hamilton we should suppress this entirely from non-debug builds ^

Sorry this is a core add-on bug, filed mozilla-rally/rally-core-addon#582 for that.

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

No branches or pull requests

1 participant