-
Notifications
You must be signed in to change notification settings - Fork 20
Open
Description
Routing to /verify-captcha reveals the following, which is not OK for production:
TypeError: Cannot call method 'hasOwnProperty' of undefined
at appendFormValue (/bundle/bundle/programs/server/npm/meteorhacks_sikka/node_modules/request/request.js:463:17)
at Request.init (/bundle/bundle/programs/server/npm/meteorhacks_sikka/node_modules/request/request.js:477:11)
at new Request (/bundle/bundle/programs/server/npm/meteorhacks_sikka/node_modules/request/request.js:264:8)
at request (/bundle/bundle/programs/server/npm/meteorhacks_sikka/node_modules/request/index.js:50:10)
at Function.request.post (/bundle/bundle/programs/server/npm/meteorhacks_sikka/node_modules/request/index.js:80:27)
at Object._processCaptcha (packages/meteorhacks_sikka/packages/meteorhacks_sikka.js:361:1)
at _verifyCaptchaMiddleware (packages/meteorhacks_sikka/packages/meteorhacks_sikka.js:342:1)
at doCall (packages/meteorhacks_picker/packages/meteorhacks_picker.js:106:1)
at [object Object].PickerImp._processRoute (packages/meteorhacks_picker/packages/meteorhacks_picker.js:100:1)
at processNextRoute (packages/meteorhacks_picker/packages/meteorhacks_picker.js:68:1)
Suggested fix:
//in Sikka.routes._processCaptcha middleware
var captchResponse = parsedUrl.query['g-recaptcha-response'];
//add this check to make sure the query param exists
if (!captchResponse){
return next();
}
...
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels