View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001059 | Xdebug | Tracing | public | 2014-07-03 09:06 | 2020-03-12 16:49 |
Reporter | RQuadling | Assigned To | derick | ||
Priority | normal | Severity | feature | Reproducibility | N/A |
Status | closed | Resolution | fixed | ||
Target Version | 2.6.0 | Fixed in Version | 2.6.0beta1 | ||
Summary | 0001059: Add filter capabilities to tracing, stack traces, and code coverage. | ||||
Description | Hi. The call trace feature of xDebug is brilliant. But sometimes it can be very very long, especially when used with a third party framework. It would be really nice to be able either/both include/exclude namespaces from the trace, so for an application based upon Laravel, xDebug could be setup to not trace into the Illuminate namespace, this reducing the trace log to just my code. Things could get messy where the first call is to the Illuminate code and then at some point the framework calls my code which is where the tracing is required. Not really got much of a clue as to what else could be done around this, but I would certainly find it very useful to have such a feature. | ||||
Tags | No tags attached. | ||||
Operating System | |||||
PHP Version | 5.5.5-5.5.9 | ||||
|
I would like to do this, but not just for tracing. I am intending to add filtering based on either a list of directory prefixes, or a list of namespaces. With this filter enabled, this would act for tracing, stacktraces, code coverage, and perhaps also profiling (the latter is trickier, as it currently collates times from different functions into one blob). |
|
That's great. I'm not really able to contribute to the code, but can offer testing support if you need it. |
|
Can't wait for this one! It would drastically speed up my current functional testing with coverage report. By the way, to make gathering of coverage easier in functional testing context (like JaCoCo) it would help if there were an option to activate code coverage for any request without the need to call to xdebug_start_coverage(). I did a patch to PHP to call those function for me automatically and save the report in a compatible format with PHPUnit tool to merge report but it is very dirty and the lack of blacklist/whitelist in xdebug make the all process slow... |
|
Progress on leveraging this for php-code-coverage (and PHPUnit) is tracked at https://github.com/sebastianbergmann/php-code-coverage/issues/514 |
|
I recently made a commit to implement filters in a basic way. This The branch is: https://github.com/derickr/xdebug/tree/issue1059-filter Currently, the API for this is a single function: proto void xdebug_set_filter(int group, int type, array filters) The group is either one of XDEBUG_FILTER_TRACING, XDEBUG_FILTER_PROFILER, or The type is one of XDEBUG_PATH_WHITELIST, XDEBUG_PATH_BLACKLIST, Right now, if the filter finds a specific frame as "filtered out", it does |
|
I would say yes to filtering cascades to child calls. But each call would need to check the appropriate list. So, if we reach a call that would be filtered out by the blacklist, it is filtered out. Any calls that are subsequently made would be checked against the whitelist and if appropriate, filtered in. And then you would look at the blacklist to filter out any further child calls. etc. |
|
Right now, you can't set both a black list or a white list. It's going to one of each. Remember that it costs time to check against these lists as well. I have to choose upon function call whether this function matches the blacklist or whitelist and act accordingly. And I'm inclined to make the filters cascading. |
|
I can't make out the exact meaning of "you can't set both a black list or a white list. It's going to one of each.". Does this mean you can only set a blacklist OR a whitelist? The main issue I would have with the filters cascading is that, for example, using a framework like Laravel, the code I've written is called upon by Laravel, not the other way around. Laravel sets up the routes, gathers all the data before calling my controllers. The controllers interact with Laravel's Eloquent ORM which in turn calls upon my models, observers, views and presenters. All VERY interleaved. If I excluded ./vendor, pretty much the second line of executed code would be within either composer's autoload or Laravel's framework. As for checking against the list on each call, can you not add a semaphore to the file upon loading, indicating that the contents is filtered in or out? That way, the list has already been processed and it is a (hopefully) simple semaphore check to report the call? (I am not an expert here, so I may be WILDLY out). Maybe it would be a good idea if you could demonstrate tracing of a Laravel based application but without reporting any of the Laravel calls. |
|
Making the filter cascading looks weird to me. The common use case for code coverage filtering is to filter out the vendor folder, keeping only your own code. |
|
I won't be able to actually experiment with the current state of the implementation until next week, but on first glance the API looks okay to me. PHPUnit used to support both whitelisting and blacklisting for code coverage filtering. By default it used to use a blacklist and when a whitelist was configured it would use the whitelist but not the blacklist. This blacklist XOR whitelist usage makes sense to me. If it were up to me, though, I would not bother adding blacklist support to Xdebug. What I need from PHPUnit's perspective, whitelisting, is there and I will use that. I am not sure that I understand the discussion about cascading correctly. How I expect the filtering for code coverage to work is this: when I put one directory on the whitelist then only files that are in/under that directory must show up in the code coverage data. It does not matter where code in that files is called from. |
|
Has this feature had any movement? |
|
Closed in Xdebug master, for Xdebug 2.6.0 (and beta1). |
Date Modified | Username | Field | Change |
---|---|---|---|
2014-07-03 09:06 | RQuadling | New Issue | |
2016-12-13 21:30 | derick | Note Added: 0004048 | |
2016-12-13 21:30 | derick | Assigned To | => derick |
2016-12-13 21:30 | derick | Status | new => confirmed |
2016-12-13 21:30 | derick | Target Version | => 2.6.0dev |
2016-12-13 21:30 | derick | Summary | Allow tracing with inclusion/exclusion of namespaces. => Add filter capabilities to tracing, stack traces, and code coverage. |
2016-12-13 21:34 | derick | Relationship added | has duplicate 0000365 |
2016-12-13 21:35 | derick | Relationship added | has duplicate 0000504 |
2016-12-14 23:14 | RQuadling | Note Added: 0004073 | |
2017-01-20 03:50 | jboffel | Note Added: 0004187 | |
2017-03-22 15:02 | sebastian | Note Added: 0004240 | |
2017-04-25 11:02 | derick | Note Added: 0004324 | |
2017-04-25 14:47 | RQuadling | Note Added: 0004325 | |
2017-04-25 14:57 | derick | Note Added: 0004326 | |
2017-04-25 23:56 | RQuadling | Note Added: 0004327 | |
2017-04-26 06:14 | stof | Note Added: 0004328 | |
2017-04-26 10:53 | sebastian | Note Added: 0004329 | |
2017-07-31 10:46 | RQuadling | Note Added: 0004387 | |
2017-12-02 18:35 | derick | Target Version | 2.6.0dev => 2.6.0 |
2017-12-28 19:14 | derick | Note Added: 0004527 | |
2017-12-28 19:14 | derick | Status | confirmed => closed |
2017-12-28 19:14 | derick | Resolution | open => fixed |
2017-12-28 19:14 | derick | Fixed in Version | => 2.6.0 |
2017-12-28 19:16 | derick | Fixed in Version | 2.6.0 => 2.6.0beta1 |
2020-03-12 16:49 | derick | Category | Feature/Change request => Tracing |