`add` and `remove` are not async safe about webtorrent

[ThaUnknown]

those are async functions, you simply need to run

await client.add()

and

await client.remove()

[uriva]

consider the case where add is called from different places in the code. If there is an sync context switch inside add, then we can be inside it twice.

await is not equivalent to a lock.

[ThaUnknown]

fair

[uriva]

while we're at it, these functions should probably not have a cb parameter and be async at the same time.

[ThaUnknown]

they should be, but not as a part of this issue

this was planned for ages: https://github.com/orgs/webtorrent/projects/1#card-60702582

they became async because other issues were being blocked by it, and using .then would be useless if it was later re-written to await anyways, and dropping callback based calls for webtorrent, means doing it inside close to 20 packages, which requries a lot of testing and rewriting

[github-actions]

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?