Adds Recaptcha V2 support while keeping backward compatibility

[felipou]

I got the version 2 implementation from hgGeorg's fork (https://github.com/hgGeorg/node-recaptcha/blob/master/lib/recaptcha.js)
and inserted it here but keeping the lib backwards compatible so
it is possible to make a smooth transition, or even support both
versions in the same code.

[superclarkk]

Please can someone let me know when this is likely to be merged? Thanks

[Sommerfeld]

is it possible that u might mix up some callback parameters in the verify method? (success, errorMessage)

[felipou]

@Sommerfeld Sorry, I didn't understand what you want. Also, I didn't write any of this code, I just merged the codes for both versions.

[Sommerfeld]

/lib/v2.js

Recaptcha.prototype.verify = function(callback) {
    var self = this;

    // See if we can declare this invalid without even contacting Recaptcha.
    if (typeof(this.data) === 'undefined') {
        this.error_code = 'verify-params-incorrect';
        return callback('verify-params-incorrect', false); // (errorMessage, succes)
    }
    if (!('remoteip' in this.data &&
          'response' in this.data))
    {
        this.error_code = 'verify-params-incorrect';
        return callback(false, 'verify-params-incorrect'); // (success, errorMessage)
    }
    if (this.data.response === '') {
        this.error_code = 'incorrect-captcha-sol';
        return callback('incorrect-captcha-sol', false); // (errorMessage, succes)
    }
   ...
};

If i'm correct after checking /lib/v1.js it should always be (succes, errorMessage). It's not important for me just got confused when i read the code. So if I'm wrong just ignore me:)

[superclarkk]

Just in case anyone needs it, I'm using recaptcha2 now, and it works really well.