8000 Symfony Profiler Broken in 2.8+ · Issue #22671 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

Symfony Profiler Broken in 2.8+ #22671

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.

Sign up for GitHub

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

Jump to bottom
Closed
alexgurrola opened this issue May 8, 2017 · 4 comments
Closed

Symfony Profiler Broken in 2.8+ #22671

alexgurrola opened this issue May 8, 2017 · 4 comments
Labels
HttpKernel

Comments

@alexgurrola
Copy link
Q A
Bug report? yes
Feature request? no
BC Break report? yes
RFC? no
Symfony version 2.8+

We have a system of Load Balancers and Web Servers, so we use Database Storage for the Profiler, which has somehow become deprecated and removed in 2.8+, which will keep us from ever upgrading in the future. Is there any alternatives for retrieving this logic or will we be forced to create our own extensions for future use?

Location:

https://github.com/symfony/symfony/tree/2.8/src/Symfony/Component/HttpKernel/Profiler
@javiereguiluz
Copy link
Member

You are right. We deprecated all profiler storages in #15944. I don't think they will be introduced in the future, so the solution is the one you suggested: create your own profiler storage. In this case, it could be as simple as copying the original Symfony database storage that you used before.

alexgurrola added a commit to Sitetheory/ProfilerStorageBundle that referenced this issue May 8, 2017
@alexgurrola
Initial Port from Symfony
0411358 
This is in regards to Symfony Issue #22671
symfony/symfony#22671
@alexgurrola
Copy link
Author

@javiereguiluz Thanks for your help. We decided to maintain a copy of the ProfilerStorage Handlers for the foreseeable future. If there's any other advice going forward, we're all ears. 😄

@alexgurrola
Copy link
Author

Update: We created a composer package with the necessary code to keep these ProfilerStorage Handlers easily installed, configured, and maintained. The package is located here: ProfilerStorageBundle

@weaverryan
Copy link
Member

Thanks @alexgurrola!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
HttpKernel
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@javiereguiluz @weaverryan @alexgurrola
0