diff options
author | Hannah von Reth <hannah.vonreth@owncloud.com> | 2020-03-18 14:26:15 +0300 |
---|---|---|
committer | Kevin Ottens <kevin.ottens@nextcloud.com> | 2020-12-15 12:59:08 +0300 |
commit | 868b05f25b0a80c21b9a44b22b8b5b3dc7bc5d6c (patch) | |
tree | dc0035f65ef58dd19e6b95c9d5fc1a4a5dcadc92 /src/common/vfs.cpp | |
parent | d63d4cdf623d82ebe32e51f336f5395f395af5fe (diff) |
Improve logging of issues during plugin loading
If the plugin could not be loaded the client calls qFatal
Make the loading warnings critical so they get printed before we crash
Diffstat (limited to 'src/common/vfs.cpp')
-rw-r--r-- | src/common/vfs.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/common/vfs.cpp b/src/common/vfs.cpp index 69c377170..45a00ef8b 100644 --- a/src/common/vfs.cpp +++ b/src/common/vfs.cpp @@ -185,26 +185,26 @@ std::unique_ptr<Vfs> OCC::createVfsFromPlugin(Vfs::Mode mode) auto pluginPath = pluginFileName("vfs", name); if (!isVfsPluginAvailable(mode)) { - qCWarning(lcPlugin) << "Could not load plugin: not existant or bad metadata" << pluginPath; + qCCritical(lcPlugin) << "Could not load plugin: not existant or bad metadata" << pluginPath; return nullptr; } QPluginLoader loader(pluginPath); auto plugin = loader.instance(); if (!plugin) { - qCWarning(lcPlugin) << "Could not load plugin" << pluginPath << loader.errorString(); + qCCritical(lcPlugin) << "Could not load plugin" << pluginPath << loader.errorString(); return nullptr; } auto factory = qobject_cast<PluginFactory *>(plugin); if (!factory) { - qCWarning(lcPlugin) << "Plugin" << pluginPath << "does not implement PluginFactory"; + qCCritical(lcPlugin) << "Plugin" << loader.fileName() << "does not implement PluginFactory"; return nullptr; } auto vfs = std::unique_ptr<Vfs>(qobject_cast<Vfs *>(factory->create(nullptr))); if (!vfs) { - qCWarning(lcPlugin) << "Plugin" << pluginPath << "does not create a Vfs instance"; + qCCritical(lcPlugin) << "Plugin" << loader.fileName() << "does not create a Vfs instance"; return nullptr; } |