Allow applications to specify their own logger instance

[richvdh]

Allow clients to specify a logger instance when instantiating a matrix client via the ClientOpts.

This is is particularly useful for clients that want to support multiple matrix accounts, but I also want to use it in the cypress tests.

It builds on the work done by @clokep in #3484, and replaces that PR.

It's not yet complete - there are still lots of places that use the global logger - but it puts the framework in place and catches quite a lot of the main culprits.

Fixes #1899.

---

Here's what your changelog entry will look like:

✨ Features

• Allow applications to specify their own logger instance (#3792). Fixes Ability to have a a logger per MatrixClient #1899.

[florianduros]

Oh, it was put back in draft, didn't see it in my notification. Ignore my review.

[richvdh]

@florianduros I put it back in draft while I fixed the tests, which I've now done - are you still happy?

I don't think it's going to be practical to meet the coverage gate on this PR since it hits a range of existing code with poor coverage.

[florianduros]