Fix deleteKeychainEntries: Stay consistent with job->setInsecureFallback(false)
authorMichael Schuster <michael@schuster.ms>
Sun, 8 Dec 2019 00:59:50 +0000 (01:59 +0100)
committerMichael Schuster <48932272+misch7@users.noreply.github.com>
Sun, 8 Dec 2019 01:47:22 +0000 (02:47 +0100)
We never fall back to the insecure variant in the whole codebase, so don't do it here.

Signed-off-by: Michael Schuster <michael@schuster.ms>
src/gui/creds/webflowcredentials.cpp

index 25da56783f299fbf1146c44a59a3d69f6759161a..008e119f376dec473fa6f1bcbaf90ecc1501c6e4 100644 (file)
@@ -691,7 +691,7 @@ void WebFlowCredentials::deleteKeychainEntries(bool oldKeychainEntries) {
     auto startDeleteJob = [this, oldKeychainEntries](QString user) {
         DeletePasswordJob *job = new DeletePasswordJob(Theme::instance()->appName());
         addSettingsToJob(_account, job);
-        job->setInsecureFallback(true);
+        job->setInsecureFallback(false);
         job->setKey(keychainKey(_account->url().toString(),
                                 user,
                                 oldKeychainEntries ? QString() : _account->id()));