Merge pull request #26 from CodeLieutenant/chore/keys

Refactor Keys **BREAKING**

Author: CodeLieutenant

benchmarks/EncryptionBench.php

@@ -62,7 +62,7 @@ public function providerChaCha20Crypto(): Generator
             foreach ($encoders as $encoderName => $encoder) {
                 yield 'ChaCha20-' . $encoderName . '-' . $dataName => [
                     'data' => $dataValue,
-                    'encrypter' => new XChaCha20Poly1305Encrypter(new KeyLoader(Random::bytes(32)), $logger, $encoder),
+                    'encrypter' => new XChaCha20Poly1305Encrypter(new KeyKeyLoader(Random::bytes(32)), $logger, $encoder),
                 ];
             }
         }
@@ -91,7 +91,7 @@ public function providerAESGCMCrypto(): Generator
             foreach ($encoders as $encoderName => $encoder) {
                 yield 'ChaCha20-' . $encoderName . '-' . $dataName => [
                     'data' => $dataValue,
-                    'encrypter' => new AesGcm256Encrypter(new KeyLoader(Random::bytes(32)), $logger, $encoder),
+                    'encrypter' => new AesGcm256Encrypter(new KeyKeyLoader(Random::bytes(32)), $logger, $encoder),
                 ];
             }
         }

benchmarks/KeyLoader.php renamed to benchmarks/KeyKeyLoader.php

@@ -4,9 +4,9 @@
 
 namespace CodeLieutenant\LaravelCrypto\Benchmarks;
 
-use CodeLieutenant\LaravelCrypto\Keys\Loader;
+use CodeLieutenant\LaravelCrypto\Contracts\KeyLoader;
 
-class KeyLoader implements Loader
+class KeyKeyLoader implements KeyLoader
 {
     public function __construct(
         private readonly string $key

src/Console/GenerateCryptoKeysCommand.php

@@ -5,7 +5,7 @@
 namespace CodeLieutenant\LaravelCrypto\Console;
 
 use CodeLieutenant\LaravelCrypto\Keys\Generators\AppKeyGenerator;
-use CodeLieutenant\LaravelCrypto\Keys\Generators\Blake2bHashingKeyGenerator;
+use CodeLieutenant\LaravelCrypto\Keys\Generators\Blake2BHashingKeyGenerator;
 use CodeLieutenant\LaravelCrypto\Keys\Generators\EdDSASignerKeyGenerator;
 use CodeLieutenant\LaravelCrypto\Keys\Generators\HmacKeyGenerator;
 use Exception;
@@ -31,7 +31,7 @@ class GenerateCryptoKeysCommand extends Command
     public function handle(
         EdDSASignerKeyGenerator $edDSAGenerator,
         AppKeyGenerator $appKeyGenerator,
-        Blake2bHashingKeyGenerator $blake2bKeyGenerator,
+        Blake2BHashingKeyGenerator $blake2bKeyGenerator,
         HmacKeyGenerator $hmacKeyGenerator,
     ): int {
         $show = $this->option('show');

src/Contracts/KeyGeneration.php

@@ -2,9 +2,9 @@
 
 declare(strict_types=1);
 
-namespace CodeLieutenant\LaravelCrypto\Keys\Generators;
+namespace CodeLieutenant\LaravelCrypto\Contracts;
 
-interface Generator
+interface KeyGenerator
 {
     public function generate(?string $write): ?string;
 }

src/Keys/Generators/Generator.php renamed to src/Contracts/KeyGenerator.php

@@ -2,9 +2,9 @@
 
 declare(strict_types=1);
 
-namespace CodeLieutenant\LaravelCrypto\Keys;
+namespace CodeLieutenant\LaravelCrypto\Contracts;
 
-interface Loader
+interface KeyLoader
 {
     public function getKey(): string|array;
 }

src/Keys/Loader.php renamed to src/Contracts/KeyLoader.php

@@ -4,25 +4,24 @@
 
 namespace CodeLieutenant\LaravelCrypto\Encryption;
 
-use Exception;
 use CodeLieutenant\LaravelCrypto\Contracts\Encoder;
-use CodeLieutenant\LaravelCrypto\Contracts\KeyGeneration;
+use CodeLieutenant\LaravelCrypto\Contracts\KeyLoader;
 use CodeLieutenant\LaravelCrypto\Encoder\JsonEncoder;
-use CodeLieutenant\LaravelCrypto\Keys\Loader;
 use CodeLieutenant\LaravelCrypto\Support\Base64;
 use CodeLieutenant\LaravelCrypto\Traits\Crypto;
+use Exception;
 use Illuminate\Contracts\Encryption\DecryptException;
 use Illuminate\Contracts\Encryption\Encrypter;
 use Illuminate\Contracts\Encryption\EncryptException;
 use Illuminate\Contracts\Encryption\StringEncrypter;
 use Psr\Log\LoggerInterface;
 
-final class AesGcm256Encrypter implements Encrypter, KeyGeneration, StringEncrypter
+final class AesGcm256Encrypter implements Encrypter, StringEncrypter
 {
     use Crypto;
 
     public function __construct(
-        private readonly Loader $keyLoader,
+        private readonly KeyLoader $keyLoader,
         private readonly Encoder $encoder = new JsonEncoder(),
         private readonly ?LoggerInterface $logger = null,
     ) {
@@ -70,11 +69,6 @@ public function decrypt($payload, $unserialize = true)
         };
     }
 
-    public static function generateKey(string $cipher): string
-    {
-        return sodium_crypto_aead_aes256gcm_keygen();
-    }
-
     public static function nonceSize(): int
     {
         return SODIUM_CRYPTO_AEAD_AES256GCM_NPUBBYTES;

src/Encryption/AesGcm256Encrypter.php

@@ -5,9 +5,8 @@
 namespace CodeLieutenant\LaravelCrypto\Encryption;
 
 use CodeLieutenant\LaravelCrypto\Contracts\Encoder;
-use CodeLieutenant\LaravelCrypto\Contracts\KeyGeneration;
+use CodeLieutenant\LaravelCrypto\Contracts\KeyLoader;
 use CodeLieutenant\LaravelCrypto\Encoder\JsonEncoder;
-use CodeLieutenant\LaravelCrypto\Keys\Loader;
 use CodeLieutenant\LaravelCrypto\Support\Base64;
 use CodeLieutenant\LaravelCrypto\Traits\Crypto;
 use Exception;
@@ -17,12 +16,12 @@
 use Illuminate\Contracts\Encryption\StringEncrypter;
 use Psr\Log\LoggerInterface;
 
-final class XChaCha20Poly1305Encrypter implements Encrypter, KeyGeneration, StringEncrypter
+final class XChaCha20Poly1305Encrypter implements Encrypter, StringEncrypter
 {
     use Crypto;
 
     public function __construct(
-        private readonly Loader $keyLoader,
+        private readonly KeyLoader $keyLoader,
         private readonly Encoder $encoder = new JsonEncoder(),
         private readonly ?LoggerInterface $logger = null,
     ) {
@@ -81,11 +80,6 @@ public function decrypt($payload, $unserialize = true)
         return $decrypted;
     }
 
-    public static function generateKey(string $_): string
-    {
-        return sodium_crypto_aead_xchacha20poly1305_ietf_keygen();
-    }
-
     public static function nonceSize(): int
     {
         return SODIUM_CRYPTO_AEAD_XCHACHA20POLY1305_IETF_NPUBBYTES;

src/Encryption/XChaCha20Poly1305Encrypter.php

@@ -4,14 +4,15 @@
 
 namespace CodeLieutenant\LaravelCrypto\Keys\Generators;
 
+use CodeLieutenant\LaravelCrypto\Contracts\KeyGenerator;
 use CodeLieutenant\LaravelCrypto\Encryption\AesGcm256Encrypter;
 use CodeLieutenant\LaravelCrypto\Encryption\XChaCha20Poly1305Encrypter;
 use CodeLieutenant\LaravelCrypto\Enums\Encryption;
 use CodeLieutenant\LaravelCrypto\Traits\EnvKeySaver;
 use Illuminate\Contracts\Config\Repository;
 use Illuminate\Encryption\Encrypter;
 
-class AppKeyGenerator implements Generator
+class AppKeyGenerator implements KeyGenerator
 {
     use EnvKeySaver;
 
@@ -31,8 +32,8 @@ public function generate(?string $write): ?string
 
         $new = $this->formatKey(
             match (Encryption::tryFrom($cipher)) {
-                Encryption::SodiumAES256GCM => AesGcm256Encrypter::generateKey($cipher),
-                Encryption::SodiumXChaCha20Poly1305 => XChaCha20Poly1305Encrypter::generateKey($cipher),
+                Encryption::SodiumAES256GCM => sodium_crypto_aead_aes256gcm_keygen(),
+                Encryption::SodiumXChaCha20Poly1305 => sodium_crypto_aead_xchacha20poly1305_ietf_keygen(),
                 default => Encrypter::generateKey($cipher),
             }
         );