I understand that `from_config_sync` and `SyncMessageSender` don't need to take in publisher instead of a client, but maybe they should for consistency with their async counterparts?