On Thu, Dec 12 2013, José Luis Di Biase wrote:

>>      db.admin.command(getParameterObj, function(err, result) {
>> -        if(err) { logger.error(err); return; }
>> +        if(err) { logger.error('Error: ' + err); return error(err); }
>
> In this case we are logging twice this error.
> Maybe We can remove keep this line ``` if(err) { return error(err);
> }```

we are only logging it twice if user doesn't override error.
maybe the default error handler should message that you should override
it.

I thought it was important to log the error even if the user decides to
not log in it's error handler.

Cheers,
-- 
Niv Sardi
/*
 * Free Software hacker & hacktivist
 */