Merge pull request #1679 from tarlepp/feat/string-component

Feat - Added `symfony/string` component and refactored `ApiKey` entity to use it

composer.json

@@ -54,6 +54,7 @@
 		"symfony/runtime": "6.0.3",
 		"symfony/security-bundle": "6.0.3",
 		"symfony/serializer": "6.0.3",
+		"symfony/string": "6.0.3",
 		"symfony/translation": "6.0.3",
 		"symfony/twig-bundle": "6.0.3",
 		"symfony/validator": "6.0.3",

src/Entity/ApiKey.php

@@ -23,13 +23,13 @@
 use Ramsey\Uuid\UuidInterface;
 use Symfony\Bridge\Doctrine\Validator\Constraints as AssertCollection;
 use Symfony\Component\Serializer\Annotation\Groups;
+use Symfony\Component\String\ByteString;
 use Symfony\Component\Validator\Constraints as Assert;
 use Throwable;
 use function array_map;
 use function array_merge;
 use function array_unique;
 use function array_values;
-use function random_int;
 
 /**
  * Class ApiKey
@@ -165,14 +165,7 @@ public function setToken(string $token): self
      */
     public function generateToken(): self
     {
-        $random = '';
-        $chars = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
-
-        for ($i = 0; $i < 40; $i++) {
-            $random .= $chars[random_int(0, 61)];
-        }
-
-        return $this->setToken($random);
+        return $this->setToken(ByteString::fromRandom(40)->toString());
     }
 
     public function getDescription(): string