Problem/Motivation

When CustomElementsViewSubscriber redirects to the back end, it prepends a slash to the request URI. Request URIs should already begin with a slash, but there is a separate bug I discovered over in #3336148: Add support for drupal forms in the frontend where the slash is incorrectly stripped when the custom elements subrequest is processed.

As it stands this code currently works, but if the linked patch is committed then this will need to be updated here as well.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 3357157-2.patch917 byteslongwave

Comments

longwave created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Needs review
StatusFileSize
new 3357157-2.patch917 bytes

In combination with the patch from #3336148: Add support for drupal forms in the frontend this provides consistent URLs (no double slashes, or missing path components) when switching between the backend and frontend on forms or via redirects.

fago’s picture

fago
Primary language German
Location Vienna
commented
Title: Request URIs should already begin with a slash » Redirect Request URIs already begin with a slash

Right. Clarifying issue title that this only affects the redirects.

fago’s picture

fago
Primary language German
Location Vienna
commented
Title: Redirect Request URIs already begin with a slash » Redirect request URIs should already begin with a slash

  • fago committed 756ceb96 on 2.x authored by longwave 
    Issue #3357157 by longwave: Redirect request URIs should already begin...
fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs review » Fixed

Indeed. Tested - it needs #3363234: Subrequests request URI must have a leading slash to stay working, so we'll move on with that separately.

Status: Fixed » Closed (fixed)

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