Project

General

Profile

Actions

Bug #82515

closed

sys_domain.domainName must allow more tha 80 chars.

Added by Alexander Schnitzler over 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
-
Start date:
2017-09-20
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
7.0
Tags:
Complexity:
no-brainer
Is Regression:
Sprint Focus:

Description

domainName is a VARCHAR, which is an unnecessary low limit nowadays.

RFC 1035, section 2.3.4 (page 9) specifies a length limit of 255 chars: https://tools.ietf.org/html/rfc1035

Actions #1

Updated by Patrik Karisch over 6 years ago

  • Priority changed from Should have to Must have
  • Target version changed from 9.0 to Candidate for patchlevel
  • TYPO3 Version changed from 9 to 8

I set the TYPO3 version to 8 :) I hope it's patchworthy for v8. As this is affecting current projects, we have domains which are near the limit on platform.sh feature environments. If we would have a longer domain in the routes the resulting domain would exceed 80 chars, which is easily possible.

Actions #2

Updated by Patrik Karisch over 6 years ago

  • TYPO3 Version changed from 8 to 7

From feedback on twitter, it should go into each maintained version :)

Actions #3

Updated by Gerrit Code Review over 6 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54186

Actions #4

Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54189

Actions #5

Updated by Georg Ringer over 6 years ago

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

Updated by Gerrit Code Review over 6 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54191

Actions #7

Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54191

Actions #8

Updated by Georg Ringer over 6 years ago

  • Status changed from Under Review to Resolved
Actions #9

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF