LavaMoat / snow

Use Snow to finally secure your web app's same origin realms!
https://lavamoat.github.io/snow/demo/
MIT License
102 stars 9 forks source link

Snow breaks the spec regarding {add/remove}EventListener #13

Closed weizman closed 2 years ago

weizman commented 2 years ago

There are specific rules to follow when adding/removing event listeners, some of them are:

  1. a listener removed must be the same one that was added
  2. a listener added won't apply again if was added before (unless options are different)

Snow breaks that because it adds a wrapped version of the provided listener but doesn't track it for when (1) or (2) happen.