Bug 59989 - [patch] Add log message when dav_auto_checkin fails
Summary: [patch] Add log message when dav_auto_checkin fails
Status: NEW
Alias: None
Product: Apache httpd-2
Classification: Unclassified
Component: All (show other bugs)
Version: 2.5-HEAD
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ---
Assignee: Apache HTTPD Bugs Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-11 15:40 UTC by jiazhouyang
Modified: 2016-08-11 15:55 UTC (History)
0 users



Attachments
Add log message when dav_auto_checkin fails (3.76 KB, patch)
2016-08-11 15:40 UTC, jiazhouyang
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description jiazhouyang 2016-08-11 15:40:45 UTC
Created attachment 34135 [details]
Add log message when dav_auto_checkin fails

Function dav_auto_checkin may fail. The majority of calls have checked the return value and printed log message like the following code snippet, while the others have not. Those unchecked ones probably need logs too (see attachment).

/* httpd-2.4.10/modules/dav/main/mod_dav.c:3421:11 */
    err = dav_auto_checkin(r, resource, 0 /*undo*/, 0 /*unlock*/, &av_info);
    if (err != NULL) {
        /* just log a warning */
        err = dav_push_error(r->pool, err->status, 0,
                             "The VERSION-CONTROL was successful, but there "
                             "was a problem automatically checking in "
                             "the parent collection.",
                             err);
        dav_log_err(r, err, APLOG_WARNING);
    }