Closed Bug 643890 Opened 13 years ago Closed 13 years ago

Use Apache's ErrorLog when using mod_perl/vhosts

Categories

(Bugzilla :: Administration, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 4.2

People

(Reporter: idl0r, Assigned: idl0r)

References

()

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (X11; Linux x86_64; rv:2.0) Gecko/20110322 Firefox/4.0
Build Identifier: 

All errors will be logged in apache's global error log instead of the ErrorLog of the specified vhost. See URL for further information.

Reproducible: Always
Status: UNCONFIRMED → NEW
Ever confirmed: true
We're using this patch currently
Attachment #521259 - Flags: review?(mkanat)
Comment on attachment 521259 [details] [diff] [review]
0001-Use-apache-s-ErrorLog.patch

Review of attachment 521259 [details] [diff] [review]:
-----------------------------------------------------------------

::: mod_perl.pl
@@ +56,5 @@
>  use Bugzilla::Template ();
>  use Bugzilla::Util ();
>  
> +use Apache2::Log ();
> +*CORE::GLOBAL::warn = \&Apache2::ServerRec::warn;

Looks good. We should put the CORE::GLOBAL::warn in a BEGIN block, though. That can be done on checkin.
Attachment #521259 - Flags: review?(mkanat) → review+
It's *sort* of a bug that log messages go to the wrong place. At least 4.2 should get this fix.
Flags: approval+
OS: Linux → All
Hardware: x86_64 → All
Target Milestone: --- → Bugzilla 4.2
Assignee: administration → idl0r
Status: NEW → ASSIGNED
mkanat, as you reviewed the patch and a fix on checkin is needed, could you please commit this patch yourself?
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                       
modified mod_perl.pl
Committed revision 7860.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: