Merge pull request #185 from vtsykun/fix/pat-token

Fix PAT token is not works with usernames with capital letter
vtsykun · Oct 28, 2023 · a24d4d1 · a24d4d1
2 parents 8f4c96e + 07bb18b
commit a24d4d1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Model/BaseUser.php b/src/Model/BaseUser.php
@@ -294,7 +294,7 @@ public function setUsername($username)
      */
     public function setUsernameCanonical($usernameCanonical)
     {
-        $this->usernameCanonical = $usernameCanonical;
+        $this->usernameCanonical = mb_strtolower($usernameCanonical);
 
         return $this;
     }

diff --git a/src/Repository/UserRepository.php b/src/Repository/UserRepository.php
@@ -52,7 +52,7 @@ public function findOneByUsernameOrEmail(string $usernameOrEmail)
             }
         }
 
-        return $this->findOneBy(['usernameCanonical' => $usernameOrEmail]);
+        return $this->findOneBy(['usernameCanonical' => mb_strtolower($usernameOrEmail)]);
     }
 
     public function getPackageMaintainersQueryBuilder(Package $package, User $excludeUser=null)
-Original file line number
+Diff line change
@@ Expand Up @@
                 }
             }
-            return $this->findOneBy(['usernameCanonical' => $usernameOrEmail]);
+            return $this->findOneBy(['usernameCanonical' => mb_strtolower($usernameOrEmail)]);
         }
         public function getPackageMaintainersQueryBuilder(Package $package, User $excludeUser=null)
@@ Expand Down @@