Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[WebProfilerBundle] Route for _profiler is Symfony specific #6949

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
henrikbjorn opened this issue Feb 2, 2013 · 0 comments
Closed

[WebProfilerBundle] Route for _profiler is Symfony specific #6949

henrikbjorn opened this issue Feb 2, 2013 · 0 comments
Labels
Bug WebProfilerBundle

Comments

@henrikbjorn
Copy link
Contributor

When trying to use the Profiler and WebDebugToolbar outside of the fullstack framework the _profiler route's _controller is too specific as it uses FrameworkBundle:Redirect:redirect instead of a service as the other routes.

Current routes.

+--------------------------+-----------+------+-----------------------------------+---------------------------------------------------------+
| Name                     | Method    | Host | Path                              | Controller                                              |
+--------------------------+-----------+------+-----------------------------------+---------------------------------------------------------+
| _profiler_search         | ANY       |      | /_profiler/search                 | web_profiler.controller.profiler:searchAction           |
| _profiler_search_bar     | ANY       |      | /_profiler/search_bar             | web_profiler.controller.profiler:searchBarAction        |
| _profiler_purge          | ANY       |      | /_profiler/purge                  | web_profiler.controller.profiler:purgeAction            |
| _profiler_info           | ANY       |      | /_profiler/info/{about}           | web_profiler.controller.profiler:infoAction             |
| _profiler_import         | ANY       |      | /_profiler/import                 | web_profiler.controller.profiler:importAction           |
| _profiler_export         | ANY       |      | /_profiler/export/{token}.txt     | web_profiler.controller.profiler:exportAction           |
| _profiler_phpinfo        | ANY       |      | /_profiler/phpinfo                | web_profiler.controller.profiler:phpinfoAction          |
| _profiler_search_results | ANY       |      | /_profiler/{token}/search/results | web_profiler.controller.profiler:searchResultsAction    |
| _profiler                | ANY       |      | /_profiler/{token}                | web_profiler.controller.profiler:panelAction            |
| _profiler_router         | ANY       |      | /_profiler/{token}/router         | web_profiler.controller.router:panelAction              |
| _profiler_exception      | ANY       |      | /_profiler/{token}/exception      | web_profiler.controller.exception:showAction            |
| _profiler_exception_css  | ANY       |      | /_profiler/{token}/exception.css  | web_profiler.controller.exception:cssAction             |
| _profiler_redirect       | ANY       |      | /_profiler/                       | FrameworkBundle:Redirect:redirect                       |
| _wdt                     | ANY       |      | /_wdt/{token}                     | web_profiler.controller.profiler:toolbarAction          |
+--------------------------+-----------+------+-----------------------------------+---------------------------------------------------------+
@fabpot fabpot mentioned this issue Feb 15, 2013
Merged
fabpot added a commit that referenced this issue Feb 17, 2013
@fabpot
merged branch fabpot/profiler-fix (PR #7079)
9f0beab 
This PR was merged into the 2.2 branch.

Commits
-------

a9f60f7 [WebProfilerBundle] removed dependency on FrameworkBundle (closes #6949)

Discussion
----------

[WebProfilerBundle] removed dependency on FrameworkBundle (closes #6949)

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #6949
| License       | MIT
| Doc PR        | n/a
@fabpot fabpot closed this as completed in a9f60f7 Feb 18, 2013
fabpot added a commit that referenced this issue Feb 18, 2013
@fabpot
Merge branch '2.2'
1c783f6 
* 2.2: (22 commits)
  [Process] Fix regression introduced in #6620 / 880da01, fixes #7082
  [HttpKernel] added a unit for the previous commit (closes #7025)
  [HttpFoundation] fixed, overwritten CONTENT_TYPE
  [BrowserKit] fixed test added in the previous merge (refs #7059)
  [FrameworkBundle] tweaked reference dumper command (see #7093)
  Remove unnecessary comment and change test name
  [Config] tweaked dumper to indent multi-line info
  [HttpKernel] added some tests for previous merge
  Fix REMOTE_ADDR for cached subrequests
  [FrameworkBundle] CSRF should be on by default
  [WebProfilerBundle] removed dependency on FrameworkBundle (closes #6949)
  [HttpKernel] added error display suppression when using the ErrorHandler (if not, errors are displayed twice, refs #6254)
  [HttpFoundation] tweaked previous merge
  [HttpFoundation] Added getter for httpMethodParameterOverride state
  Create validators.lv.xlf
  [Process] Warn user with a useful message when tmpfile() failed
  [BrowserKit] added a test to make sure HTTP authentication is preserved when submitting a form
  Remove array type hint from GetResponseForControllerResultEvent::setControllerResult()
  bumped Symfony version to 2.2.0-DEV
  Revert "merged branch povilas/issue_6101 (PR #6708)"
  ...
ostrolucky pushed a commit to ostrolucky/symfony that referenced this issue Mar 25, 2018
@fabpot
[WebProfilerBundle] removed dependency on FrameworkBundle (closes sym…
5922dd0 
…fony#6949)
ostrolucky pushed a commit to ostrolucky/symfony that referenced this issue Mar 25, 2018
@fabpot
Merge branch '2.2'
47dca74 
* 2.2: (22 commits)
  [Process] Fix regression introduced in symfony#6620 / 880da01, fixes symfony#7082
  [HttpKernel] added a unit for the previous commit (closes symfony#7025)
  [HttpFoundation] fixed, overwritten CONTENT_TYPE
  [BrowserKit] fixed test added in the previous merge (refs symfony#7059)
  [FrameworkBundle] tweaked reference dumper command (see symfony#7093)
  Remove unnecessary comment and change test name
  [Config] tweaked dumper to indent multi-line info
  [HttpKernel] added some tests for previous merge
  Fix REMOTE_ADDR for cached subrequests
  [FrameworkBundle] CSRF should be on by default
  [WebProfilerBundle] removed dependency on FrameworkBundle (closes symfony#6949)
  [HttpKernel] added error display suppression when using the ErrorHandler (if not, errors are displayed twice, refs symfony#6254)
  [HttpFoundation] tweaked previous merge
  [HttpFoundation] Added getter for httpMethodParameterOverride state
  Create validators.lv.xlf
  [Process] Warn user with a useful message when tmpfile() failed
  [BrowserKit] added a test to make sure HTTP authentication is preserved when submitting a form
  Remove array type hint from GetResponseForControllerResultEvent::setControllerResult()
  bumped Symfony version to 2.2.0-DEV
  Revert "merged branch povilas/issue_6101 (PR symfony#6708)"
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug WebProfilerBundle
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@henrikbjorn @fabpot