Commit c816bff

committed

prevent empty username on dbauth register endpoint

1 parent cd78c8c commit c816bffCopy full SHA for c816bff

File tree

-0

lines changed

src/Tqdev/PhpCrudApi
- Middleware
  - DbAuthMiddleware.php
- Record
  - ErrorCode.php

-0

lines changed

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -73,6 +73,9 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface`
`73`	`73`	`if (!$registerUser) {`
`74`	`74`	`return $this->responder->error(ErrorCode::AUTHENTICATION_FAILED, $username);`
`75`	`75`	`}`
	`76`	`+ if(strlen(trim($username)) == 0){`
	`77`	`+ return $this->responder->error(ErrorCode::USERNAME_EMPTY, $username);`
	`78`	`+ }`
`76`	`79`	`if (strlen($password) < $passwordLength) {`
`77`	`80`	`return $this->responder->error(ErrorCode::PASSWORD_TOO_SHORT, $passwordLength);`
`78`	`81`	`}`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,7 @@ class ErrorCode`
`33`	`33`	`const PAGINATION_FORBIDDEN = 1019;`
`34`	`34`	`const USER_ALREADY_EXIST = 1020;`
`35`	`35`	`const PASSWORD_TOO_SHORT = 1021;`
	`36`	`+ const USERNAME_EMPTY = 1022;`
`36`	`37`
`37`	`38`	`private $values = [`
`38`	`39`	`0000 => ["Success", ResponseFactory::OK],`
`@@ -58,6 +59,7 @@ class ErrorCode`
`58`	`59`	`1019 => ["Pagination forbidden", ResponseFactory::FORBIDDEN],`
`59`	`60`	`1020 => ["User '%s' already exists", ResponseFactory::CONFLICT],`
`60`	`61`	`1021 => ["Password too short (<%d characters)", ResponseFactory::UNPROCESSABLE_ENTITY],`
	`62`	`+ 1022 => ["Username is empty or only whitespaces", ResponseFactory::UNPROCESSABLE_ENTITY],`
`61`	`63`	`9999 => ["%s", ResponseFactory::INTERNAL_SERVER_ERROR],`
`62`	`64`	`];`
`63`	`65`

Comments

(0)