Bug 1847096 Comment 12 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

It should be possible to pass a service worker identifier to each notification so that the command would include it as a parameter. I'm not quite sure we should do that, because:

* I'm not sure it's exactly safe, as some malware could scan the notification center and craft a command with the identifier to mock a user click, which then would run JS with malware crafted arguments. But maybe we can't do much for such an infected machine, as there would be hundreds more evil things to do for a malware?
* Do we know the same can be done for Linux and macOS, i.e. can we store the same amount of information into libnotify and macOS notification system?
   * Can you share how you did it for Windows? Bug 1803416 only has the proxy binary part, and per my understanding we hash the information [here](https://searchfox.org/mozilla-central/source/widget/windows/ToastNotificationHandler.cpp#366-370) to fit the system limitation. My plan was to use this hashed tag everywhere, but open for better ideas!

Please feel free to correct me if I got something wrong 👍🏻
It should be possible to pass a service worker identifier to each notification so that the command would include it as a parameter. I'm not quite sure we should do that, because:

* I'm not sure it's exactly safe, as some malware could scan the notification center and craft a command with the identifier to mock a user click, which then would run JS with malware crafted arguments. But maybe we can't do much for such an infected machine, as there would be hundreds more evil things to do for a malware?
* Do we know the same can be done for Linux and macOS, i.e. can we store the same amount of information into libnotify and macOS notification system?
   * Can you share how you did it for Windows? Bug 1803416 only has the proxy binary part, and per my understanding we hash the information [here](https://searchfox.org/mozilla-central/source/widget/windows/ToastNotificationHandler.cpp#366-370) to fit the system limitation. My plan was to use this hashed tag everywhere (in other backends and notification store), but open for better ideas!

Please feel free to correct me if I got something wrong 👍🏻

Back to Bug 1847096 Comment 12