Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

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

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

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

rhelmer opened this issue May 7, 2021 · 2 comments
Assignees
Labels
ARCHIVED CLOSED at time of archiving

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.

@cknowles-admin cknowles-admin added the ARCHIVED CLOSED at time of archiving label Jul 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ARCHIVED CLOSED at time of archiving
Projects
None yet
Development

No branches or pull requests

2 participants