Passthru max UID

Author: abnegate

src/Database/Database.php

@@ -5151,6 +5151,7 @@ public function updateDocuments(
                 $indexes,
                 $this->adapter->getIdAttributeType(),
                 $this->maxQueryValues,
+                $this->adapter->getMaxUIDLength(),
                 $this->adapter->getMinDateTime(),
                 $this->adapter->getMaxDateTime(),
                 $this->adapter->getSupportForAttributes()
@@ -6710,6 +6711,7 @@ public function deleteDocuments(
                 $indexes,
                 $this->adapter->getIdAttributeType(),
                 $this->maxQueryValues,
+                $this->adapter->getMaxUIDLength(),
                 $this->adapter->getMinDateTime(),
                 $this->adapter->getMaxDateTime(),
                 $this->adapter->getSupportForAttributes()
@@ -6908,6 +6910,7 @@ public function find(string $collection, array $queries = [], string $forPermiss
                 $indexes,
                 $this->adapter->getIdAttributeType(),
                 $this->maxQueryValues,
+                $this->adapter->getMaxUIDLength(),
                 $this->adapter->getMinDateTime(),
                 $this->adapter->getMaxDateTime(),
                 $this->adapter->getSupportForAttributes()
@@ -7139,6 +7142,7 @@ public function count(string $collection, array $queries = [], ?int $max = null)
                 $indexes,
                 $this->adapter->getIdAttributeType(),
                 $this->maxQueryValues,
+                $this->adapter->getMaxUIDLength(),
                 $this->adapter->getMinDateTime(),
                 $this->adapter->getMaxDateTime(),
                 $this->adapter->getSupportForAttributes()
@@ -7204,6 +7208,7 @@ public function sum(string $collection, string $attribute, array $queries = [],
                 $indexes,
                 $this->adapter->getIdAttributeType(),
                 $this->maxQueryValues,
+                $this->adapter->getMaxUIDLength(),
                 $this->adapter->getMinDateTime(),
                 $this->adapter->getMaxDateTime(),
                 $this->adapter->getSupportForAttributes()

src/Database/Validator/Key.php

@@ -6,16 +6,6 @@
 
 class Key extends Validator
 {
-    protected bool $allowInternal = false; // If true, you keys starting with $ are allowed
-
-    /**
-     * Maximum length for Key validation
-     */
-    protected int $maxLength;
-
-    /**
-     * @var string
-     */
     protected string $message;
 
     /**
@@ -33,10 +23,10 @@ public function getDescription(): string
     /**
      * Expression constructor
      */
-    public function __construct(bool $allowInternal = false, int $maxLength = 255)
-    {
-        $this->allowInternal = $allowInternal;
-        $this->maxLength = $maxLength;
+    public function __construct(
+        private readonly bool $allowInternal = false,
+        private readonly int $maxLength = 36,
+    ) {
         $this->message = 'Parameter must contain at most ' . $this->maxLength . ' chars. Valid chars are a-z, A-Z, 0-9, period, hyphen, and underscore. Can\'t start with a special char';
     }
 
@@ -46,7 +36,6 @@ public function __construct(bool $allowInternal = false, int $maxLength = 255)
      * Returns true if valid or false if not.
      *
      * @param $value
-     *
      * @return bool
      */
     public function isValid($value): bool
@@ -59,15 +48,14 @@ public function isValid($value): bool
             return false;
         }
 
-        // no leading special characters
+        // No leading special characters
         $leading = \mb_substr($value, 0, 1);
         if ($leading === '_' || $leading === '.' || $leading === '-') {
             return false;
         }
 
         $isInternal = $leading === '$';
 
-
         if ($isInternal && !$this->allowInternal) {
             return false;
         }
@@ -83,6 +71,7 @@ public function isValid($value): bool
         if (\preg_match('/[^A-Za-z0-9\_\-\.]/', $value)) {
             return false;
         }
+
         // At most maxLength chars
         if (\mb_strlen($value) > $this->maxLength) {
             return false;

src/Database/Validator/Queries/Documents.php

@@ -2,7 +2,6 @@
 
 namespace Utopia\Database\Validator\Queries;
 
-use Exception;
 use Utopia\Database\Database;
 use Utopia\Database\Document;
 use Utopia\Database\Validator\IndexedQueries;
@@ -16,18 +15,21 @@
 class Documents extends IndexedQueries
 {
     /**
-     * Expression constructor
-     *
      * @param array<mixed> $attributes
      * @param array<mixed> $indexes
      * @param string $idAttributeType
-     * @throws Exception
+     * @param int $maxValuesCount
+     * @param \DateTime $minAllowedDate
+     * @param \DateTime $maxAllowedDate
+     * @param bool $supportForAttributes
+     * @throws \Utopia\Database\Exception
      */
     public function __construct(
         array $attributes,
         array $indexes,
         string $idAttributeType,
         int $maxValuesCount = 5000,
+        int $maxUIDLength = 36,
         \DateTime $minAllowedDate = new \DateTime('0000-01-01'),
         \DateTime $maxAllowedDate = new \DateTime('9999-12-31'),
         bool $supportForAttributes = true
@@ -60,7 +62,7 @@ public function __construct(
         $validators = [
             new Limit(),
             new Offset(),
-            new Cursor(),
+            new Cursor($maxUIDLength),
             new Filter(
                 $attributes,
                 $idAttributeType,

src/Database/Validator/Query/Cursor.php

@@ -8,6 +8,10 @@
 
 class Cursor extends Base
 {
+    public function __construct(private readonly int $maxLength = 36)
+    {
+    }
+
     /**
      * Is valid.
      *
@@ -33,7 +37,7 @@ public function isValid($value): bool
                 $cursor = $cursor->getId();
             }
 
-            $validator = new UID();
+            $validator = new UID($this->maxLength);
             if ($validator->isValid($cursor)) {
                 return true;
             }

src/Database/Validator/UID.php

@@ -7,7 +7,7 @@ class UID extends Key
     /**
      * Expression constructor
      */
-    public function __construct(int $maxLength = 255)
+    public function __construct(int $maxLength = 36)
     {
         parent::__construct(false, $maxLength);
     }