When each locale uses a separate domain, redirecting fails with this error:

Redirects to external URLs are not allowed by default, use \Drupal\Core\Routing\TrustedRedirectResponse for it.
CommentFileSizeAuthor
#6 3119466-6-disable-language-external-redirect.patch5.21 KBmr.baileys
#6 3119466-6-disable-language-external-redirect-test-only.patch3.93 KBmr.baileys
#3 3119466-3-disable-language-external-redirect.patch5.21 KBmr.baileys
#3 3119466-3-disable-language-external-redirect-test-only.patch3.93 KBmr.baileys
#2 disable_language-3119466-2.patch1.27 KBprudloff

Comments

prudloff created an issue. See original summary.

prudloff’s picture

prudloff commented
StatusFileSize
new disable_language-3119466-2.patch1.27 KB

The attached patch fixes this issue.

mr.baileys’s picture

mr.baileys
Primary language Dutch
Location 🇧🇪 (Ghent)
commented
Status: Active » Needs review
StatusFileSize
new 3119466-3-disable-language-external-redirect-test-only.patch3.93 KB
new 3119466-3-disable-language-external-redirect.patch5.21 KB

Confirmed that redirects to domains are failing. Attached is a test demonstrating the issue, and a combined patch with both the test and fix.

The last submitted patch, 3: 3119466-3-disable-language-external-redirect-test-only.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Status: Needs review » Needs work

The last submitted patch, 3: 3119466-3-disable-language-external-redirect.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

mr.baileys’s picture

mr.baileys
Primary language Dutch
Location 🇧🇪 (Ghent)
commented
Status: Needs work » Needs review
StatusFileSize
new 3119466-6-disable-language-external-redirect-test-only.patch3.93 KB
new 3119466-6-disable-language-external-redirect.patch5.21 KB

The last submitted patch, 6: 3119466-6-disable-language-external-redirect-test-only.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

  • mr.baileys authored b16e2ad on 8.x-1.x 
    Issue #3119466 by mr.baileys, prudloff: Can't redirect to a different...
thomas cys’s picture

thomas cys
Primary language Dutch
commented
Status: Needs review » Fixed

Thanks for the patch.

Status: Fixed » Closed (fixed)

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