| Conditions | 9 | 
| Paths | 8 | 
| Total Lines | 72 | 
| Lines | 0 | 
| Ratio | 0 % | 
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php | ||
| 45 | public function encrypt($data) | ||
| 46 |     { | ||
| 47 | $rsaPublicKey = $this->configuration->getPublicKey(); | ||
| 48 | |||
| 49 |         if (!is_string($data) || !is_string($rsaPublicKey)) { | ||
| 50 | // Invalid argument | ||
| 51 |             throw new InvalidArgumentException('Invalid input was provided to the encrypt method'); | ||
| 52 | } | ||
| 53 | |||
| 54 | // Use AES-256 in GCM | ||
| 55 | $symmetricAlgorithm = 'aes-256-gcm'; | ||
| 56 | |||
| 57 | // Generate initialisation vector for the symmetric encryption algorithm | ||
| 58 | $ivLength = openssl_cipher_iv_length($symmetricAlgorithm); | ||
| 59 |         if (false === $ivLength) { | ||
| 60 | // Error generating key | ||
| 61 | throw new InvalidArgumentException( | ||
| 62 | 'Unable to generate an initialization vector (iv) based on the selected symmetric encryption algorithm' | ||
| 63 | ); | ||
| 64 | } | ||
| 65 | |||
| 66 | $iv = openssl_random_pseudo_bytes($ivLength); | ||
| 67 |         if (false === $iv) { | ||
| 68 | // Error generating key | ||
| 69 |             throw new InvalidArgumentException('Unable to generate a correct initialization vector (iv)'); | ||
| 70 | } | ||
| 71 | |||
| 72 | // Generate a 256 bits AES key | ||
| 73 | $secretKey = openssl_random_pseudo_bytes(256 / 8); | ||
| 74 |         if (false === $secretKey) { | ||
| 75 | // Error generating key | ||
| 76 |             throw new InvalidArgumentException('Unable to generate the secret key'); | ||
| 77 | } | ||
| 78 | |||
| 79 | // Encrypt the data | ||
| 80 | $tag = ''; | ||
| 81 | $ciphertext = openssl_encrypt($data, $symmetricAlgorithm, $secretKey, 0, $iv, $tag); | ||
| 82 |         if (false === $ciphertext) { | ||
| 83 | // Encryption failed | ||
| 84 | throw new InvalidArgumentException( | ||
| 85 |                 sprintf('Unable to encrypt the data, ssl error: "%s"', openssl_error_string()) | ||
| 86 | ); | ||
| 87 | } | ||
| 88 | |||
| 89 | // Encrypt symmetric key | ||
| 90 | $rsaPublicKeyHandle = openssl_pkey_get_public($rsaPublicKey); | ||
| 91 |         if (false === $rsaPublicKeyHandle) { | ||
| 92 | // Reading RSA public key failed | ||
| 93 |             throw new InvalidArgumentException('Reading RSA public key failed'); | ||
| 94 | } | ||
| 95 | $encryptedKey = ''; | ||
| 96 | |||
| 97 | $res = openssl_public_encrypt($secretKey, $encryptedKey, $rsaPublicKeyHandle, OPENSSL_PKCS1_OAEP_PADDING); | ||
| 98 |         if (false === $res) { | ||
| 99 | // Key encryption failed | ||
| 100 | openssl_pkey_free($rsaPublicKeyHandle); | ||
| 101 |             throw new InvalidArgumentException('Key encryption failed'); | ||
| 102 | } | ||
| 103 | |||
| 104 | openssl_pkey_free($rsaPublicKeyHandle); | ||
| 105 | $output = json_encode( | ||
| 106 | [ | ||
| 107 | 'algorithm' => $symmetricAlgorithm, | ||
| 108 | 'iv' => base64_encode($iv), | ||
| 109 | 'tag' => base64_encode($tag), | ||
| 110 | 'ciphertext' => base64_encode($ciphertext), | ||
| 111 | 'encrypted_key' => base64_encode($encryptedKey), | ||
| 112 | ] | ||
| 113 | ); | ||
| 114 | |||
| 115 | $this->writer->write($output); | ||
| 116 | } | ||
| 117 | } | ||
| 118 |