Adds warning method to the CLI printer

Author: moebrowne

src/Backup/Crypter/OpenSSL.php

@@ -4,6 +4,7 @@
 use phpbu\App\Backup\Restore\Plan;
 use phpbu\App\Backup\Target;
 use phpbu\App\Cli\Executable;
+use phpbu\App\Result;
 use phpbu\App\Util;
 
 /**
@@ -54,6 +55,36 @@ class OpenSSL extends Abstraction implements Simulator, Restorable
      */
     private $keepUncrypted;
 
+
+    /**
+     * @inheritDoc
+     */
+    public function crypt(Target $target, Result $result)
+    {
+        if ($this->getExecutable($target)->isUsingWeakAlgorithm()) {
+            $name = strtolower(get_class($this));
+
+            $result->warn($name . ': The ' . $this->algorithm . ' algorithm is considered weak');
+        }
+
+        return parent::crypt($target, $result);
+    }
+
+
+    /**
+     * @inheritDoc
+     */
+    public function simulate(Target $target, Result $result)
+    {
+        if ($this->getExecutable($target)->isUsingWeakAlgorithm()) {
+            $name = strtolower(get_class($this));
+
+            $result->warn($name . ': The ' . $this->algorithm . ' algorithm is considered weak');
+        }
+
+        return parent::simulate($target, $result);
+    }
+
     /**
      * Setup
      *

src/Cli/Executable/OpenSSL.php

@@ -162,7 +162,7 @@ class OpenSSL extends Abstraction implements Executable
         ]
     ];
 
-    private $weakCiphers = [
+    private $weakAlgorithms = [
         'rc2'          => true,
         'rc2-40'       => true,
         'rc2-64'       => true,
@@ -291,6 +291,15 @@ public function useAlgorithm(string $algorithm): OpenSSL
         return $this;
     }
 
+    public function isUsingWeakAlgorithm(): bool
+    {
+        if (null === $this->algorithm) {
+            throw new Exception('algorithm is not set');
+        }
+
+        return isset($this->weakAlgorithms[$this->algorithm]);
+    }
+
     /**
      * Use base64 encoding
      *

src/Result/PrinterCli.php

@@ -110,6 +110,7 @@ public static function getSubscribedEvents(): array
     {
         return [
             'phpbu.debug'           => 'onDebug',
+            'phpbu.warning'         => 'onWarning',
             'phpbu.app_start'       => 'onPhpbuStart',
             'phpbu.backup_start'    => 'onBackupStart',
             'phpbu.backup_failed'   => 'onBackupFailed',
@@ -406,6 +407,16 @@ public function onDebug(Event\Debug $event)
         }
     }
 
+    /**
+     * Warnings.
+     *
+     * @param \phpbu\App\Event\Warning $event
+     */
+    public function onWarning(Event\Warning $event)
+    {
+        $this->writeWithColor('fg-black, bg-yellow', $event->getMessage() . PHP_EOL);
+    }
+
     /**
      * phpbu end event.
      *

tests/phpbu/Backup/Crypter/OpenSSLTest.php

@@ -171,4 +171,23 @@ public function testGetSuffix()
         $suffix  = $openSSL->getSuffix();
         $this->assertEquals('enc', $suffix);
     }
+
+    /**
+     * Tests that a warning is emitted for weak algorithms
+     */
+    public function testWeakAlgorithmsCauseWarnings()
+    {
+        $runner = $this->getRunnerMock();
+        $runner->expects($this->once())
+            ->method('run')
+            ->willReturn($this->getRunnerResultMock(0, 'openssl'));
+
+        $target    = $this->createTargetMock(__FILE__);
+        $appResult = $this->getAppResultMock();
+        $appResult->expects($this->once())->method('warn');
+
+        $openSSL = new OpenSSL($runner);
+        $openSSL->setup(['pathToOpenSSL' => PHPBU_TEST_BIN, 'password' => 'fooBarBaz', 'algorithm' => 'des']);
+        $openSSL->crypt($target, $appResult);
+    }
 }