WPIntell

Source evidence

registration_errors filter priority change

No CAPTCHA reCAPTCHA · support · 2018-11-28T11:39:00+00:00

complaintsentiment
highseverity
0.97relevance
0replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

6 / 32 rows with source links

18.8% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

26 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
pvankouteren unresolved
I noticed that in anr-captcha-class.php on line 44: add_filter (‘registration_errors’, array($this, ‘registration_verify’), 10, 3 ); The default priority of 10 is used. Note that other plugins operating on the registration_errors filter (like the “new user approve” plugin) also use the default priority. Validation / verification priority should be as high (so: in wp code as low int) as possible. In its current state because of equal priorities it’s a gamble: the recaptcha is there, but will it be validated before the other plugin gets the data or not? In my current case the “new user approve” plugin was installed earlier and their filter is triggered before this plugin, so the registration passes before verification is done. When I changed the priority to 9, just to test, all worked OK. I would suggest to set it even lower. -> Other verify calls should be reconsidered as well.

Comments

0 shown

No comments were stored for this source.