Bug #52311

Uuid validator doesn't have string boundaries

Added by Adrian Föder almost 2 years ago. Updated about 1 year ago.

Status:Resolved Start date:2013-09-26
Priority:Must have Due date:
Assigned To:Adrian Föder % Done:

100%

Category:Validation
Target version:-
PHP Version: Complexity:
Has patch:Yes Affected Flow version:Git 2.0

Description

\TYPO3\Flow\Validation\Validator\UuidValidator::$PATTERN_MATCH_UUID is '/([a-f0-9]){8}-([a-f0-9]){4}-([a-f0-9]){4}-([a-f0-9]){4}-([a-f0-9]){12}/' and therefore a UUID like 9e6e8e93-36b7-4158-9893-257632bc7a37sdsdflkjsdflkj is considered valid, incorrectly ofc

Associated revisions

Revision 52b18622
Added by Adrian Föder almost 2 years ago

[BUGFIX] UUID Validator rejects too long UUIDs

Now the validation regex has start- and end of string
boundaries, hence rejecting strings which only
contain a valid UUID, but having extraordinary strings
before or behind.

Change-Id: Ic912e903abd34c3319d64a6187d03d9a62c08679
Releases: master, 2.0
Fixes: #52311

Revision 0b91dd2c
Added by Adrian Föder almost 2 years ago

[BUGFIX] UUID Validator rejects too long UUIDs

Now the validation regex has start- and end of string
boundaries, hence rejecting strings which only
contain a valid UUID, but having extraordinary strings
before or behind.

Change-Id: Ic912e903abd34c3319d64a6187d03d9a62c08679
Releases: master, 2.0
Fixes: #52311

History

#1 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24076

#2 Updated by Gerrit Code Review almost 2 years ago

Patch set 1 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/24926

#3 Updated by Adrian Föder about 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF