Bug #81486

eID=dumpFile does not take config.absRefPrefix into account

Added by Xavier Perseguers over 3 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2017-06-06
Due date:
% Done:

100%

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

Description

How to reproduce

  • Enable EXT:realurl for your website
  • Ensure you don't use config.baseURL
  • Set config.absRefPrefix = / (or any other value, according to your actual configuration
  • Edit fileadmin file storage properties and set "is publicly available" to false under tab "Access capabilities"
  • From any subpage (at any level) of your website, insert an image

What happens

Since the storage folder is not publicly available, instead of having

<img src="/fileadmin/path/to/image.jpg" ...

TYPO3 generates

<img src="index.php?eID=dumpFile&...

Problem

On Apache, it looks like the rewrite configuration provided by TYPO3 seems to handle the missing "/" before "index.php" but the "/" is missing nevertheless, and may fail, e.g., on Nginx.


Related issues

Precedes TYPO3 Core - Bug #81944: Invalid asset prefix when installed in a subdirectoryClosed2017-07-24

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/53140

#2

Updated by Markus Klein over 3 years ago

Requirements for the publicUrl functionality are:

- relative and absolute URLs (external storages will deliver absolute anyway, eg S3)
- folder-awareness (docroot, absrefPrefix)
- CLI compatibility

#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/53140

#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/53140

#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/53140

#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/53140

#7

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/53140

#8

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/53140

#9

Updated by Gerrit Code Review over 3 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/53439

#10

Updated by Xavier Perseguers over 3 years ago

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

Updated by Xavier Perseguers over 3 years ago

  • Precedes Bug #81944: Invalid asset prefix when installed in a subdirectory added
#12

Updated by Gerrit Code Review over 3 years ago

  • Status changed from Resolved 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/53789

#13

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/53789

#14

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/53789

#15

Updated by Gerrit Code Review about 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/53789

#16

Updated by Gerrit Code Review about 3 years ago

Patch set 2 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/53439

#17

Updated by Helmut Hummel about 3 years ago

  • Status changed from Under Review to Resolved
#18

Updated by Benni Mack about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF