Bug #46463

jumpUrl_transferSession throws hash exception

Added by Philipp Idler over 8 years ago. Updated almost 7 years ago.

Status:
Rejected
Priority:
Should have
Category:
Frontend
Target version:
Start date:
2013-03-20
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
5.3
Tags:
Complexity:
medium
Is Regression:
Yes
Sprint Focus:

Description

This problem was also described here: #46071

Problem is solved when jumpUrl_transferSession is set to false
but sometimes this option is neccessary and can't be simply turned off.

The exception occurs for all pages with type "external_link" in TYPO3 4.5.25.

Only option right now is to go back to 4.5.23 in my opinion.


Related issues

Related to TYPO3 Core - Bug #46071: External links doesn't work anymore since recent security update 4.5.24Closed2013-03-06

Actions
#1

Updated by Ernesto Baschny over 8 years ago

  • Status changed from New to Accepted
  • Assignee set to Ernesto Baschny
  • Target version set to 4.5.26
  • Complexity set to medium

Confirmed, and I am working on a solution to this. Thanks for the report.

#2

Updated by Jens Schmietendorf over 8 years ago

Same problem with 4.6.18,
downgrade to 4.6.16 fixes it.

#3

Updated by Gerrit Code Review over 8 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19270

#4

Updated by Gerrit Code Review over 8 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/19271

#5

Updated by Ernesto Baschny over 8 years ago

Please test the patch in the review system. For your convenience I already made a review for 4.5 code, so it's easier to apply and test with 4.5 - 4.7 (pre namespace changes).

#6

Updated by Philipp Idler over 8 years ago

Patch works for 4.5.25 =)

#7

Updated by Gerrit Code Review over 8 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19270

#8

Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19270

#9

Updated by Ernesto Baschny about 8 years ago

  • Category set to 1394
  • Target version changed from 4.5.26 to next-patchlevel
#10

Updated by Gerrit Code Review about 8 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19270

#11

Updated by Ernesto Baschny about 8 years ago

  • Is Regression set to Yes
#12

Updated by Ernesto Baschny about 8 years ago

  • Category changed from 1394 to Frontend
#13

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

#14

Updated by Alexander Stehlik over 7 years ago

Simple test setup:

Typoscript:

config.jumpurl_enable = 1

page = PAGE
page.20 = TEXT
page.20.wrap = <p>|</p>
page.20.typolink.parameter = http://a.test.domain.local/testpage

Page TSConfig:

TSFE.jumpUrl_transferSession = 1

You will see a link in the Frontend, when you click on it

Before applying the patch:

You will get an error.

After applying the patch

You should be redirected to:

http://a.test.domain.local/testpage?&FE_SESSION_KEY=&lt;my&gt;-&lt;sessionkey>

#15

Updated by Ernesto Baschny over 7 years ago

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

Updated by Thomas Mammitzsch over 7 years ago

that still didn't make it into the core (tested with 4.5.34). Had to apply the patch myself :(

#17

Updated by Gerrit Code Review almost 7 years ago

  • Status changed from Resolved to Under Review

Patch set 2 for branch TYPO3_4-5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/19271

#18

Updated by Christian Kuhn almost 7 years ago

  • Status changed from Under Review to Rejected

The pending patch has just been abandoned by me from the review queue, please see http://review.typo3.org/19271 on why this has been done.

This issue here will be set to rejected for now and only touched again if people really think this is critical enough for 4.5 to fix. I'm sorry that this took so long with this outcome, but this area is very hard to follow with its whole impact and we must not risk additional problems in good-old 4.5 anymore.

Also available in: Atom PDF