Passing a logger through RealmConfig and SyncClientConfig

[kraenhansen]

What, How & Why?

This is a proposal to fix #7227.
SDKs that take a logger factory could call it to get a logger before passing it though to the SDK, the main breaking change that cannot easily be poly-filled is the removal of SyncManager::set_logger_factory.

☑️ ToDos

• 📝 Changelog update
• 🚦 Tests (or not relevant)
• C-API, if public C++ API changed
• bindgen/spec.yml, if public C++ API changed

[coveralls-official]

Pull Request Test Coverage Report for Build kraen.hansen_46

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.1%) to 91.702%

---

Totals
Change from base Build 1938:	-0.1%
Covered Lines:	232379
Relevant Lines:	253407

---

💛 - Coveralls

[jedelbo]

This is in conflict with commit b8aad6b, so I would like @tgoyne to comment,

[tgoyne]

We don't need it to be a logger factory, but we do expose a way to set the logger after a SyncManager is created in our public API and so need that functionality to remain until our next major version bump.

[kraenhansen]

Closing as it's unlikely this will ever merge.