See #3025752: Support adding a destination to the domain traversal link.

Adding that patch allows us to use the destination parameter in the traversal link to redirect users to a specific page on another domain, after login.

CommentFileSizeAuthor
#7 dvg-domain_traversal_with_destination-3025753-7.patch727 bytespaulvandenburg
#5 dvg-domain_traversal_with_destination-3025753-5.patch633 bytesjoshahubbers
#2 dvg-domain_traversal_with_destination-3025753-2.patch549 bytespaulvandenburg

Comments

paulvandenburg created an issue. See original summary.

paulvandenburg’s picture

paulvandenburg commented
Status: Active » Needs review
StatusFileSize
new dvg-domain_traversal_with_destination-3025753-2.patch549 bytes

This patch adds #3025752-2: Support adding a destination to the domain traversal link.

joshahubbers’s picture

joshahubbers commented
Status: Needs review » Reviewed & tested by the community
joshahubbers’s picture

joshahubbers commented
Status: Reviewed & tested by the community » Needs review
joshahubbers’s picture

joshahubbers commented
StatusFileSize
new dvg-domain_traversal_with_destination-3025753-5.patch633 bytes

rerolled against latest dev

bskibinski’s picture

bskibinski
Primary language Dutch
Location Breda, The Netherlands
commented
Status: Needs review » Reviewed & tested by the community

Patch runs on several ACC servers without hiccups. And has been deployed to production without any issues.

paulvandenburg’s picture

paulvandenburg commented
StatusFileSize
new dvg-domain_traversal_with_destination-3025753-7.patch727 bytes

Rerolled again because it didnt apply.

  • paulvandenburg committed 18ca386 on 7.x-1.x 
    Issue #3025753 by paulvandenburg, JoshaHubbers: Patch domain_traversal...
paulvandenburg’s picture

paulvandenburg commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.