Change cache key for getIdentityFlagsFromApi (#62)

* Change cache key for getIdentityFlagsFromApi

getIdentityFlagsFromApi should not be using the Global cache key prefix

* 🔧 chore(composer.json): enable php-http/discovery plugin to allow automatic discovery of HTTP clients
🐛 fix(Flagsmith.php): add cache key generation for identities to improve caching efficiency
🐛 fix(IdentitiesGenerator.php): add method to generate cache key for identities based on identifier and traits to ensure uniqueness

Author: tm1000

composer.json

@@ -30,5 +30,10 @@
             "Flagsmith\\": "src",
             "FlagsmithTest\\": "tests"
         }
+    },
+    "config": {
+        "allow-plugins": {
+            "php-http/discovery": true
+        }
     }
 }

src/Flagsmith.php

@@ -430,8 +430,9 @@ private function getIdentityFlagsFromApi(string $identifier, ?object $traits): F
     {
         try {
             $data = IdentitiesGenerator::generateIdentitiesData($identifier, $traits);
+            $cacheKey = IdentitiesGenerator::generateIdentitiesCacheKey($identifier, $traits);
             $apiFlags = $this->cachedCall(
-                'Global',
+                $cacheKey,
                 'POST',
                 $this->identities_url,
                 $data

src/Utils/IdentitiesGenerator.php

@@ -19,4 +19,10 @@ public static function generateIdentitiesData(string $identifier, ?object $trait
 
         return $identities;
     }
+
+    public static function generateIdentitiesCacheKey(string $identifier, ?object $traits)
+    {
+        $hashedTraits = $traits !== null ? '.'.sha1(serialize($traits)) : '';
+        return 'Identity.'.$identifier.$hashedTraits;
+    }
 }