Undefined array key "path" error coming from dynamically generated redirect paths

[chrismattix]

We use Sentry to monitor errors on WP sites, and we keep getting a warning coming from the canonical.php file line 619 for "Undefined array key "path"". This warning triggers at random times, and seems to be related to the dynamic nature of these paths. Is there a way to update or alter the canonical.php file to get this error to go away?

[johnjamesjacoby]

This may happen if $redirect_url is "seriously malformed" causing the parse_url() calls on either line 578 or 597 to return false.

It also may have revealed itself due to error suppression being removed as part of #49980.

I think the easiest fix is to redo what already happens at the top of redirect_canonical(). Patch imminent.

[johnjamesjacoby]

In 60496:

Canonical: prevent possible "Undefined array key" PHP error in redirect_canonical(), if the path or query array keys have gone missing.

This commit checks for the existence of (and re-adds if necessary) the path and query array keys after the $redirect variable has potentially been replaced by a second call to parse_url().

This may happen when redirecting back to the root domain, without any path ('/example/) or query (?example=1`) to parse.

Props chrismattix, dhruvang21.

Fixes #63733.

[SirLouen]

@johnjamesjacoby have you seen #63316 by any chance? (it also includes host in the mix)