[reSIProcate] Overhead in ServerAuthManager

[Daniel Pocock]

Hi,

I've introduced some more new virtual functions to ServerAuthManager.hxx:

      typedef enum AuthFailureReason
      {
           InvalidRequest,      // some aspect of the request (e.g. nonce)
                                // is not valid/tampered with
           BadCredentials,      // credentials didn't match
           Error                // processing/network error
      };
      virtual void onAuthSuccess(const SipMessage& msg);
      virtual void onAuthFailure(AuthFailureReason reason, const 
SipMessage& msg);

At various points in ServerAuthManager.cxx, these functions are called, 
so that application specific code in a derived class can log the failure 
reason, do diagnostics, etc.

I realise that not everyone might want this functionality.  Does anyone 
feel that calls like this create enough processing overhead to justify 
use of #ifdef or similar mechanisms to selectively enable this behaviour?

Regards,

Daniel