Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Debug toolbar is gone #36836

Closed
derrabus opened this issue May 16, 2020 · 7 comments
Closed

Debug toolbar is gone #36836

derrabus opened this issue May 16, 2020 · 7 comments

Comments

@derrabus
Copy link
Member

derrabus commented May 16, 2020

Symfony version(s) affected: 4.4.9-dev, 5.0.9-dev, 5.1.0-RC1

Description
I'm testing my pet projects with Symfony 5.1 since the BETA1 release. I've upgraded them to RC1 today and the first thing I've noticed in each of them is that the debug toolbar is not displayed anymore. I can still access the /_profiler/ route, though. Rolling back to BETA1 solves the issue for me.

Update: The issue also happens on 4.4/5.0 if I switch to the latest dev packages.

How to reproduce

symfony new --full --version=next my_project
cd my_project
symfony serve

Possible Solution
Display the toolbar again. 😃

@derrabus derrabus added the Bug label May 16, 2020
@derrabus derrabus changed the title Debug toolbar is gone in 5.1.0-RC1 Debug toolbar is gone May 16, 2020
@derrabus
Copy link
Member Author

The issue seems to go away if I revert #36789.

cc @marcw

@COil
Copy link
Contributor

COil commented May 16, 2020

I confirm, it's in 5.1 RC1 too.

@ovrflo
Copy link
Contributor

ovrflo commented May 16, 2020

Yep. I was just debugging this as well. The debug led me here.
I fixed it locally by changing the WebDebugToolbar priority to Profiler+1.

While I didn't quite have the issue fixed in #36789, I did know that because of the priorities we had in the past, some things might escape the profiler. That is a legitimate bug and should be fixed.

A better solution than reverting would be to update the Profiler listener priority, though right now I'm wondering if anything else would be broken by changing the toolbar's priority.

@derrabus
Copy link
Member Author

Status: Reviewed

@derrabus
Copy link
Member Author

I guess, @carsonbot won't let me update my own review status. ;-)

@derrabus
Copy link
Member Author

A better solution than reverting would be to update the Profiler listener priority,

I was searching for a quick fix, so I proposed a revert. If you've found a combination of priorities that works, feel free to submit a PR. I'd be happy to close mine then. 😃

though right now I'm wondering if anything else would be broken by changing the toolbar's priority.

Yeah, that's what I thought as well. Fiddling with listener priorities is always a dangerous thing to do.

@marcw
Copy link
Contributor

marcw commented May 17, 2020

Sorry for introducing this regression! I created an issue for my original problem.

@fabpot fabpot closed this as completed May 18, 2020
fabpot added a commit that referenced this issue May 18, 2020
This PR was merged into the 4.4 branch.

Discussion
----------

[HttpKernel] Bring back the debug toolbar

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #36836
| License       | MIT
| Doc PR        | N/A

This PR effectively reverts #36789 in order to fix a regression caused by that PR.

Commits
-------

9f8d225 Revert "Change priority of KernelEvents::RESPONSE subscriber"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants