Ticket #638 (closed enhancement: fixed)

Opened 2 months ago

Last modified 1 month ago

option to log the url associated with an error

Reported by: bibo Assigned to: bibo
Priority: normal Milestone: Jelix 1.1 beta 1
Component: jelix:core Version: trunk
Severity: normal Keywords:
Cc: Php version:
Review: review+ Hosting Provider:
Documentation needed: 0 Blocking:

Description

IT would be sometime usefull, and i mean really usefull to know which url caused an error.

Imagine an 'ill-formed content' lost in a myriad of similar content. It is a waste of time to search and find the ill one. Having the url would mean such a search become a breeze.

Attachments

jelix-trunk-#638.patch (1.3 kB) - added by bibo on 07/01/08 11:45:21.
support of %url% in messageLogFormat
jelix-trunk-#638.2.patch (1.5 kB) - added by bibo on 07/02/08 12:18:57.
some nit corrections

Change History

07/01/08 11:45:21 changed by bibo

  • attachment jelix-trunk-#638.patch added.

support of %url% in messageLogFormat

07/01/08 11:48:31 changed by bibo

  • owner changed from laurentj to bibo.
  • status changed from new to assigned.

Patch attached adds support of %url% to messageLogFormat. I've chosen to expose all request params instead of the url string. I think it is more useful to bug hunting.

Just a nit, I've used var_export but as a result message is multi-lined. Let me know if it's a problem or not.

07/01/08 11:50:48 changed by bibo

  • review set to review?.

07/02/08 12:18:57 changed by bibo

  • attachment jelix-trunk-#638.2.patch added.

some nit corrections

07/02/08 14:04:27 changed by bballizlife

I'll give you a "review+", but i don't know if i'm allowed to do so ;)

07/07/08 11:39:43 changed by laurentj

  • review changed from review? to review+.
  • milestone set to Jelix 1.1 beta 1.

It's ok for me.

07/07/08 17:25:17 changed by bibo

  • status changed from assigned to closed.
  • resolution set to fixed.
Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed