Stefan Zabka

TaskManager should check that StorageController and MPLogger are alive

1

Make this part of the watchdog?

Reimplement non relaxed shutdown

2

With the storage rewrite we lost the ability to quickly save out data in case of an imminent shutdown. We should restore this capability as our shutdowns now usually take...

Remove our custom interface types and replace them with the @types/firefox-webext-browser ones

16

Since the official types now have a lot more type definitions we can get rid of our own WebRequestOnBeforeSendHeadersEventDetails. This way we no longer need to maintain them manually. We...

We can't currently test the WebExtension on it's own. This issue aims to collect thoughts on how this could be done. For integration with CI have a look at the...

Callstack Instrument fails to capture some parts of the stack

13

@hadiamjad pointed out on Matrix that the callstack instrument captures less than the Firefox debugger shows.  

This means both testing that all interrupted visits get saved into the `interrupted_visits` table as well as making sure that the callback gets the correct bool.

Currently the TaskManager is responsible for writing into the `site_visits` table: https://github.com/mozilla/OpenWPM/blob/8dad9fd2129f8375575813baaa19992efdde47de/openwpm/task_manager.py#L381-L391 [Steven and I](https://github.com/mozilla/OpenWPM/pull/753#discussion_r548018150) agreed that it would be better to move this writing into the Storage Controller when...

This would be a wrapper around client socket providing utility methods for the four types of messages the storage controller understands. These are - append_table(table_name, record) - initialize (visit_id) -...

`manual_test.py` is one of ways we enable contributors to run an interactive session and debug the WebExtension. However since there are no tests it is constantly broken. We'd need to...

Commands can't be defined in __main__ module

3

Due to limitations in dill, classes defined in the `__main__` module can't be pickled. Since we send all of our commands through a multiprocess.queue which uses dill internally we can't...