[WebProfilerBundle] add "http status" column in the "Search Results" list #12944
Labels
Comments
|
I like the idea, since I also came in situations where I need to pick a specific failed request from the list. I'll provide a pull request for that. |
|
👍 |
|
👍 |
fabpot
added a commit
that referenced
this issue
Jan 25, 2015
…profiler search result (xelaris) This PR was squashed before being merged into the 2.7 branch (closes #13034). Discussion ---------- [HttpKernel] [WebProfilerBundle] added HTTP status to profiler search result | Q | A | ------------- | --- | Bug fix? | no | New feature? | yes | BC breaks? | yes | Deprecations? | no | Tests pass? | yes | Fixed tickets | #12944 | License | MIT | Doc PR | With this PR HTTP status codes are provided in the search results to simplify identification of particular requests. For the schema-less storage implementations (memcache(d), redis, mongodb) and the default file storage, it should work without purging existing profiles. But the code could be simplified, if it is an option to expect old profiles to be purged. For database driven storages (mysql, sqlite), the `sf_profiler_data` table must be dropped anyway (if the table isn't altered manually by adding the `status_code` column). Thus the changes are not fully BC.  Commits ------- 34ecda5 [HttpKernel] [WebProfilerBundle] added HTTP status to profiler search result
ostrolucky
pushed a commit
to ostrolucky/symfony
that referenced
this issue
Mar 25, 2018
…tus to profiler search result (xelaris) This PR was squashed before being merged into the 2.7 branch (closes symfony#13034). Discussion ---------- [HttpKernel] [WebProfilerBundle] added HTTP status to profiler search result | Q | A | ------------- | --- | Bug fix? | no | New feature? | yes | BC breaks? | yes | Deprecations? | no | Tests pass? | yes | Fixed tickets | symfony#12944 | License | MIT | Doc PR | With this PR HTTP status codes are provided in the search results to simplify identification of particular requests. For the schema-less storage implementations (memcache(d), redis, mongodb) and the default file storage, it should work without purging existing profiles. But the code could be simplified, if it is an option to expect old profiles to be purged. For database driven storages (mysql, sqlite), the `sf_profiler_data` table must be dropped anyway (if the table isn't altered manually by adding the `status_code` column). Thus the changes are not fully BC.  Commits ------- 34ecda5 [HttpKernel] [WebProfilerBundle] added HTTP status to profiler search result
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Consider adding the "http status" directly in the table list of the "Search Results" in the SymfonyProfiler to identify quickly the issue (500, 401, etc.), maybe after the "Method" column.
I am working with another developper ; he is using the API i developped. His app launchs several calls by page ; it is hard to find which request fails in this list.
The text was updated successfully, but these errors were encountered: