]> git.evergreen-ils.org Git - OpenSRF.git/commit
Consolidate duplicate osrfMethodVerifyContext handler
authorBill Erickson <berick@esilibrary.com>
Wed, 17 Oct 2012 19:57:14 +0000 (15:57 -0400)
committerBill Erickson <berick@esilibrary.com>
Thu, 8 Nov 2012 17:48:36 +0000 (12:48 -0500)
commitaca8b94f48acf751899a8a3f670a9555cf423872
tree530dc8da892b46c527743544e30e4a77e74763e7
parentbce36cb30a4ae24e2e2dcdc77a60a40e8b261d0d
Consolidate duplicate osrfMethodVerifyContext handler

Log redaction was not occuring in some cases because the code was using
the OSRF_METHOD_VERIFY_CONTEXT macro instead of the similarly named
function which does the same thing.  This change points the macro at the
function so that all code uses the same underlying code.

Note this change turns on CALL param logging unconditionally, whereas
users of the macro would previously have been able to avoid CALL logging
via the OSRF_LOG_PARAMS variable.  In practice, little code uses the
macro and all code enables OSRF_LOG_PARAMS.  If we need to add this
control back, it can be added directly to osrfMethodVerifyContext().
For now, it's one less env variable we need to define.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Dan Scott <dan@coffeecode.net>
include/opensrf/osrf_application.h