View Issue Details

WikiRelated ChangesetsJump to Notes
IDProjectCategoryView StatusDate SubmittedLast Update
0010039mantisbtbugtrackerpublic2009-01-09 17:322009-01-15 11:25
Reporterberbo Assigned Tograngeway  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
PlatformIntelOSLinuxOS VersionUbuntu 8.04
Product Version1.1.6 
Fixed in Version1.2.0a3 
Summary0010039: config_get in log_event
Description

this line in function log_event ( in logging_api.php)
$t_now = date( config_get( 'complete_date_format' ) );

can cause recursion, if log_event is called from some of the authentication routines.

It should be changed to config_get_global.

The symptom on my end: clicking on a link, I get a popup that says 'what do you want to do with this file'.

Steps To Reproduce

I discovered this when trying to use custom logging of the authentication process.

  1. constant_inc.php: LOG_MAUTH = 16
    (can't use LOG_AUTH, apparently that's taken by PHP itself).

  2. config_inc.php: $g_log_level = LOG_MAUTH;

  3. in function auth_is_cookie_valid, add this line:
    log_event(LOG_MAUTH, ' got here ');

TagsNo tags attached.

Activities

grangeway

grangeway

2009-01-09 18:15

reporter   ~0020570

fixed in git trunk

Paul

Related Changesets

MantisBT: master cd4490f6

2009-01-09 18:14

Paul Richards


Details Diff		 Fix 0010039: config_get in log_event Affected Issues
0010039
mod - core/logging_api.php Diff File

MantisBT: master 7f168ee2

2009-01-09 18:14

Paul Richards


Details Diff		 Fix 0010039: config_get in log_event Affected Issues
0010039
mod - core/logging_api.php Diff File