Project

General

Profile

Actions

Bug #92655

closed

checking for external URLs may cause link checking to never end

Added by Sybille Peters over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Linkvalidator
Target version:
-
Start date:
2020-10-21
Due date:
% Done:

100%

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

Description

I just had this in production. I am starting to increase page depth of scheduler tasks. One of the scheduler tasks never ended. Also, I noticed that several scheduler tasks were running.

So, I traced this down to an URL where the loading never ends. Why this is the case there, I don't know exactly. But is also not so relevant here.

This is a problem, because once only a single link in the site points to such a site, linkvalidator checks will not work, several scheduler tasks will run and eat up resources.

Reproduce

  1. Create a PHP page with infinite loop, e.g.
<?php
while (true) {

}

  1. create a link to to this page
  2. either start a scheduler task with linkvalidator or use "Check links"

The link checking or scheduler task will not finish.

Actions #1

Updated by Gerrit Code Review over 3 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/c/Packages/TYPO3.CMS/+/66241

Actions #2

Updated by Sybille Peters over 3 years ago

  • Description updated (diff)
Actions #3

Updated by Gerrit Code Review over 3 years ago

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

Actions #4

Updated by Gerrit Code Review over 3 years ago

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

Actions #5

Updated by Gerrit Code Review over 3 years ago

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

Actions #6

Updated by Gerrit Code Review over 3 years ago

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

Actions #7

Updated by Sybille Peters over 3 years ago

It is already possible to set the timeout with $GLOBALS['TYPO3_CONF_VARS']['HTTP']['timeout'].

So if anyone is experiencing this problem, I would recommend to set the timeout there. By default it is 0, which means no timeout.

I think it is a problem to have no timeout by default. I have created a patch which sets it via TSConfig, but you can default to using the Global Configuration, if the TSconfig is unset: https://review.typo3.org/c/Packages/TYPO3.CMS/+/66241

Actions #8

Updated by Gerrit Code Review over 3 years ago

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

Actions #9

Updated by Gerrit Code Review over 3 years ago

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

Actions #10

Updated by Gerrit Code Review over 3 years ago

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

Actions #11

Updated by Sybille Peters over 3 years ago

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

Updated by Gerrit Code Review over 3 years ago

  • Status changed from Resolved to Under Review

Patch set 2 for branch 10.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66759

Actions #13

Updated by Sybille Peters over 3 years ago

  • Status changed from Under Review to Resolved
Actions #14

Updated by Benni Mack over 3 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF