Optional logging?

When a request 404s superagent will say “Error: Not Found”, and isn’t kind enough to tell us what it didn’t find… In our applications we have had to copy/paste node-wpapi’s own transport API to add logging that tells us the URL of a request before it dispatches.

Ideally this could be option within the library itself, e.g. debug option that defaults to false and is picked up in transport methods like this:

function _httpGet( wpreq, callback, debug ) {
    if ( debug ) {
        console.log( '[node-wpapi] GET ' + wpreq.toString() );
    }
    
    checkMethodSupport( 'get', wpreq );
    // ...
}

Are you happy for me to put in a PR for this?

[sdgluck]

@amitsarangi No, please go ahead!

[kadamwhite]

@sdgluck That’s a pretty good idea, I’d definitely be interested in adding this. I’d probably want to read the “debug” option off of the WPReq instance, so that it can be set for a full bound WPAPI instance or for a specific instantiated WPRequest; I’d also lean towards giving the transport itself a debug() method that can be injected so that users can self-select for console.log, or something like morgan that could integrate with their server logs, etc.