Task #87073

Optimize error message of handlePageError()

Added by Peter Kraume 9 days ago. Updated 5 days ago.

Status:
Resolved
Priority:
Should have
Assignee:
Category:
Frontend
Start date:
2018-12-04
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
Sprint Focus:

Description

If fetching of an error page, e.g. the 404 page, fails, there is only a misleading error message:
Error handler could not fetch error page: Possible recursion detected.

This can for example happen, if you have a self signed SSL certificate in your local development environment and your site configuration defines a TYPO3 page as content source like this:

errorHandling:
  -
    errorCode: '404'
    errorHandler: Page
    errorContentSource: 't3://page?uid=10'

When fetching the content of the specified error page fails due to the self signed SSL certificate the above mentioned error message is misleading.

Another question is, if it is fine that error pages won't work in local development environments with self signed certificates.


Related issues

Related to TYPO3 Core - Bug #86935: Cannot fetch 404 page with simple base New 2018-11-15

Associated revisions

Revision 3cc23f90 (diff)
Added by Peter Kraume 5 days ago

[TASK] Display exact error message of handlePageError()

Show getUrl() error message when fetching content of error page.

Resolves: #87073
Releases: master
Change-Id: I9b2d548fc1d9a4bd07ea56c7d5776447c0c596d2
Reviewed-on: https://review.typo3.org/59026
Reviewed-by: Josef Glatz <>
Tested-by: TYPO3com <>
Reviewed-by: Joerg Kummer <>
Tested-by: Joerg Kummer <>
Tested-by: Josef Glatz <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

History

#1 Updated by Gerrit Code Review 9 days 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/59026

#2 Updated by Peter Kraume 9 days ago

  • Related to Bug #86935: Cannot fetch 404 page with simple base added

#3 Updated by Gerrit Code Review 9 days 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/59026

#4 Updated by Gerrit Code Review 6 days 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/59026

#5 Updated by Gerrit Code Review 6 days 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/59026

#6 Updated by Gerrit Code Review 6 days 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/59026

#7 Updated by Gerrit Code Review 6 days 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/59026

#8 Updated by Peter Kraume 5 days ago

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

Also available in: Atom PDF