node.js - Validating captcha along with passport.js -


is possible verify if google recaptcha succeeded before calling passport.js authenticate function?

i'm getting stuck between choosing 1 or other because both use asynchronous callbacks verify , can't next them within each other.

    function verifyrecaptcha(key, rq, rs, pa, callback) {     https.get('https://www.google.com/recaptcha/api/siteverify?secret=' + secret + '&response=' + key, function (res) {         var data = '';         res.on('data', function (chunk) {             data += chunk.tostring();         });         res.on('end', function () {             try {                 var parseddata = json.parse(data); //                    return true;                 callback(parseddata.success, rq, rs, pa);             } catch (e) { //                    return false;                 callback(false, rq, rs, pa);             }         });     }); }      app.post('/auth/signin', can_access.if_not_logged_in(), passport.setuplocalstrategy(),     function (req, res) {         console.log('here');          verifyrecaptcha(req.body['g-recaptcha-response'], function (success) {             if (success) {                 // find way signal captcha succeeded.                 return true;             } else {                 res.end('captcha failed, sorry.');                 // todo: take them previous page                 // , love of everyone, restore inputs                 return false;             }         });     },     passport.authenticate('local', {         successredirect: '/',         failureredirect: 'auth/signin',         failureflash: true     }));

i want authentication after captcha has succeeded

the way express middlewares or route handlers work executed in succession, 1 after another, long previous 1 called next()

so need call next() captcha verifying middleware passport.authenticate middleware comes next can executed.

also, if call next(err) (i.e. passing error) skip middlewares , go straight next middleware has 4 argument signature (err, req, res, next), main error handler placed @ or near end in routes/middlewares.

so try changing code this:

app.post('/auth/signin', can_access.if_not_logged_in(), passport.setuplocalstrategy(), function (req, res, next) { // <<-- 1. have next passed here     console.log('here');      verifyrecaptcha(req.body['g-recaptcha-response'], function (success) {         if (success) {             // find way signal captcha succeeded.             return next(); // <<-- 2. call next();          } else {             res.end('captcha failed, sorry.');             // todo: take them previous page             // , love of everyone, restore inputs             return false;         }     }); }, // invoked if next() called previous middleware  passport.authenticate('local', {     successredirect: '/',     failureredirect: 'auth/signin',     failureflash: true }));

Comments

Post a Comment

Popular posts from this blog

powershell Start-Process exit code -1073741502 when used with Credential from a windows service environment -

i'm running strange behavior powershell start-process call. here call: $process = start-process ` "c:\somepath\mybinary.exe" ` -passthru ` -credential $defaultcredential ` -wait ` -workingdirectory "c:\somepath" ` -loaduserprofile if ($process.exitcode -ne 0) { #do } this call return exit code - 1073741502 . after quick search, exit code seems related generic error when program not load required dll (aka. status_dll_init_failed ). when run without -credential $credential program runs correctly. in order isolate problem, manually launched some.exe in prompt target credential , runs smoothly. so problem seems come way start-process cmdlet launch process. i found potential solutions problem tried apply no luck : link , link . would have idea of what's going on here ? edit 1: run proc mon monitoring program activities when launched directly or via powershell script. problem seems occur when loading kernelbase
Read more

twig - Using Twigbridge in a Laravel 5.1 Package -

i'm trying create laravel 5.1 package uses twigbridge instead of .php views. can normal .php views display fine, when try make use of .twig templates i'm met error. possible use twigbridge templates when creating views laravel package or pretty stuck blade , php ? edit 1: loading twig template without extending works. it's when start using {% extends 'file' %} error comes out. i error error loading /virtual/laravel/packages/username/mypackage/src/views/sample.twig: template "templates.master" not defined () in "/virtual/laravel/packages/username/mypackage/src/views/sample.twig" @ line 1. my folders username |_ mypackage |_ src |_ controllers - samplecontroller.php |_ views |_ templates - master.twig - sample.twig twig template sample.twig {% extends 'templates.master' %} {% block content %} {{ parent()
Read more

c# - LINQ join Entities from HashSet's, Join vs Dictionary vs HashSet performance -

i have hashset's each stores t, have written test application compares different relation algorithms can think of, i'm not pleased results getting. does there exist more efficient ways of achieving object relations, once testing? using system; using system.collections.generic; using system.linq; using system.text; using system.threading.tasks; using system.diagnostics; namespace linqtests { class program { static void main(string[] args) { hashset<user> usertable = new hashset<user>(); hashset<userproperty> userpropertytable = new hashset<userproperty>(); #region lets create dummy data. console.writeline("please wait while creating dummy data, can take while..."); console.writeline(""); int rows = 1000000; for(int x = 0; x < rows; x++) { random rnd = new random(); // ad
Read more