WebprofilerRequestMatcher should use CurrentPathStack::getPath instead of Request::getPathInfo [#3078923]

The "Exclude" setting of the Webprofiler does not work with language prefixed paths.

I believe the WebprofilerRequestMatcher::matches method should use Drupal's CurrentPathStack::getPath instead of Symfony's Request::getPathInfo.

Comment	File	Size	Author
#4	webprofiler-use_currentpathstack_getpath-3078923-3.patch	2.07 KB	semiaddict
#2	webprofiler-use_currentpathstack_getpath-3078923-2.patch	4.24 KB	semiaddict

Comments

3 September 2019 at 12:02

semiaddict commented 3 September 2019 at 12:05

Version:	8.x-3.x-dev	» 8.x-2.x-dev
Status:	Active	» Needs work

Status	File	Size
new	webprofiler-use_currentpathstack_getpath-3078923-2.patch	4.24 KB

Attached is a patch that can be applied against the 8.x-2.x.

semiaddict commented 3 September 2019 at 12:05

Status:

Needs work

» Needs review

semiaddict commented 3 September 2019 at 12:17

Status	File	Size
new	webprofiler-use_currentpathstack_getpath-3078923-3.patch	2.07 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	webprofiler-use_currentpathstack_getpath-3078923-2.patch	4.24 KB

Fixed patch file encoding.