It is not accurate to say that you can't use external redirects with just core - TrustedRedirectResponse lets you. Admittedly, it doesn't provide the allow list that this module appears to. So it would be good to update the documentation to clarify when to use which/benefits of this over TRR and that you can use external redirects with just core.

Also, be sweet if you could update the readme/description to have a sample minimal example to help users use this.

Comments

NickWilde created an issue. See original summary.

NickDickinsonWilde’s picture

NickDickinsonWilde
He/Him/His
Primary language English
Location Victoria, BC (T'So-uke lands)
CreditAttribution: NickDickinsonWilde at BriarMoon Design commented
Issue summary: View changes
fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago at drunomics commented

You are right, that's not really clear that this is about the destination parameter.

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago at drunomics commented
Status: Active » Fixed

I update the project page accordingly, I hope it's clear now. See https://www.drupal.org/project/trusted_redirect

Status: Fixed » Closed (fixed)

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