question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Remove wheel and passive event listener feature flags

See original GitHub issue

We have an existing batch pipeline that uses the latest JavaScript Tracker along with R87 components for Enrich and Storage.

All events recorded using 2.9.1 are now going into bad rows with the following error:

{
    "errors": [
        {
            "level": "error",
            "message": "error: object instance has properties which are not allowed by the schema: [\"f_passive\",\"f_wheel\"]\n    level: \"error\"\n    schema: {\"loadingURI\":\"#\",\"pointer\":\"/items\"}\n    instance: {\"pointer\":\"/0\"}\n    domain: \"validation\"\n    keyword: \"additionalProperties\"\n    unwanted: [\"f_passive\",\"f_wheel\"]\n"
        }
    ],
}

f_passive and f_wheel seems to have been introduced by the following commit: https://github.com/snowplow/snowplow-javascript-tracker/commit/a28888111dc8ea66ca8339396bd00a2f84c4d5db#diff-3e7c17598500b655533daae15e659338R275

It is unclear what other environmental factors are necessary for the issue to appear so I am listing everything that might be relevant:

  • Enrichment runs on R87
  • Tracker is configured to POST to the collector
  • At least one other vendor uses Snowplow on the pages where we are integrated into
  • Static Iglu repositories are used for resolution (iglucentral.com and one private)

I have not yet been able to find the JSON schema that the “web” portion of the events are validated against and I am now investigating upgrade paths. It would be helpful if someone could point me to where the validation occurs so I can identify the earliest version the issue would be fixed with.

Thanks!

Issue Analytics

  • State:closed
  • Created 5 years ago
  • Comments:7 (7 by maintainers)

github_iconTop GitHub Comments

2reactions
mhadamcommented, Jul 24, 2018

I’ve got the patch done, should be able to get this out today.

0reactions
mhadamcommented, Jul 24, 2018

The new release is mirrored now.

Read more comments on GitHub >

github_iconTop Results From Across the Web

EventTarget.removeEventListener() - Web APIs | MDN
The removeEventListener() method of the EventTarget interface removes an event listener previously registered with EventTarget.
Read more >
Use passive listeners to improve scrolling performance
Learn how to improve your page's scrolling responsiveness by avoiding passive event listeners.
Read more >
Consider marking event handler as 'passive' to make the page ...
Developers can annotate touch and wheel listeners with {passive: true} to indicate that they will never invoke preventDefault. This feature shipped in ...
Read more >
Passive Event Listeners and Scroll Performance | A Faster Web
By adding a {passive: true} flag to the event listener, we can now tell the browser the listener will NOT cancel the default...
Read more >
passive event listeners #3720 - mapbox/mapbox-gl-js - GitHub
I am working with most.js stream framework to catch mapbox events in streams and stumble on this (since I got the message in...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found