bobbingwide / oik-bwtrace

debug trace for WordPress
https://www.oik-plugins.com/oik-plugins/oik-bwtrace-debug-trace-for-wordpress/
GNU General Public License v2.0
6 stars 1 forks source link

Strings containing commas in the trace summary file should be wrapped in quotes #9

Closed bobbingwide closed 9 years ago

bobbingwide commented 9 years ago

I'd written a routine to post process the trace summary output file and noticed that some transactions appeared to have taken an inordinate amount of time. Closer inspection revealed that the REQUEST_URI had contained commas.

Solution

If the request contains commas then the string should be enclosed in double quotes.

Q. What happens with double quotes in the request? A. These are encoded as %22

bobbingwide commented 9 years ago

Fixed in v2.0.6 https://github.com/bobbingwide/oik-bwtrace/commit/eba14e04e69ac285a456d143c69c03fd70b88f79