Project

General

Profile

Actions

Bug #105525

closed

Garbage in header_link field of tt_content leads to error in ReferenceIndex update

Added by Nikita Hovratov 22 days ago. Updated about 17 hours ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Link Handling & Redirect Handling
Start date:
2024-10-30
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
12
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Some editor pasted a phone number like "+491234561423523523" in the header_link field, which leads to this error:

In ExceptionConverter.php line 118:

An exception occurred while executing a query: Out of range value for column 'ref_uid' at row 1

In StatementError.php line 29:

Out of range value for column 'ref_uid' at row 1

In Statement.php line 153:

Out of range value for column 'ref_uid' at row 1

The ReferenceIndex receives a match from TypoLinkSoftReferenceParser for type db relation "pages" with value "+491234561423523523" which will be casted to an int in createEntryDataUsingRecord method, and will of course be out of range.


It was probably possible to save this value in older TYPO3 versions. In v12 inserting such a value will lead immediately to this error. So invalid links should simply be discarded on save. And for the next major version, the SoftReferenceParser / ReferenceIndex should be hardened to not fall back to default a pages relation as last resort and throw an exception instead.


Related issues 1 (0 open1 closed)

Is duplicate of TYPO3 Core - Bug #94562: Link field - insufficiently checked input leads to reference index errorResolved2021-07-15

Actions
Actions #1

Updated by Garvin Hicking 14 days ago

  • Category set to Link Handling & Redirect Handling
  • Status changed from New to Accepted
  • Priority changed from Should have to Must have
  • Target version set to Candidate for patchlevel

Raising importance on this one because when the RefIndex becomes more prominent in v14, this needs to soft-fail/convert properly IMO.

Actions #2

Updated by Christian Kuhn about 17 hours ago

  • Is duplicate of Bug #94562: Link field - insufficiently checked input leads to reference index error added
Actions #3

Updated by Christian Kuhn about 17 hours ago

  • Status changed from Accepted to Closed

Closing as dupe of #94562

Actions

Also available in: Atom PDF