Page MenuHomePhabricator
Create Task
Maniphest T416817

hCaptcha: Adjust tolerance of intermittent network errors
Open, Needs TriagePublic
Actions

Description

Summary

In T404204: Investigate options for automatic fallback to FancyCAPTCHA, we implemented a failover mode to load FancyCaptcha in the event that hCaptcha is unreachable. For reasons that are still being investigated, there are intermittent connection errors to load secure-api.js for outbound network calls (see https://logstash.wikimedia.org/goto/3638725468dcb59bc3e917335776afa1 for other examples). We currently set "failover mode" if secure-api.js is unreachable. We should instead tolerate a certain number of errors per period of time.

Acceptance criteria

  • the health checker implementation is reworked to accept a threshold of errors per window of time

Details

Related Changes in Gerrit:
SubjectRepoBranchLines +/-
ConfirmEdit: Add error count threshold for apiUrl health checksmediawiki/extensions/ConfirmEditwmf/1.46.0-wmf.14+141 -22
ConfirmEdit: Add error count threshold for apiUrl health checksmediawiki/extensions/ConfirmEditmaster+141 -22
Customize query in gerrit

Related Objects

Event Timeline

kostajh created this task.Mon, Feb 9, 5:59 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMon, Feb 9, 5:59 AM
gerritbot added a comment.Mon, Feb 9, 6:12 AM

Change #1237732 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/ConfirmEdit@master] ConfirmEdit: Add error count threshold for apiUrl health checks

https://gerrit.wikimedia.org/r/1237732

gerritbot added a project: Patch-For-Review.Mon, Feb 9, 6:12 AM
gerritbot added a comment.Mon, Feb 9, 9:08 AM

Change #1237732 merged by jenkins-bot:

[mediawiki/extensions/ConfirmEdit@master] ConfirmEdit: Add error count threshold for apiUrl health checks

https://gerrit.wikimedia.org/r/1237732

Maintenance_bot removed a project: Patch-For-Review.Mon, Feb 9, 9:31 AM
kostajh edited projects, added Product Safety and Integrity (Sprint Daffodil (Jan 19 - Feb 6)); removed Product Safety and Integrity.Mon, Feb 9, 9:36 AM
kostajh moved this task from Backlog to Needs QA on the Product Safety and Integrity (Sprint Daffodil (Jan 19 - Feb 6)) board.
ReleaseTaggerBot added a project: MW-1.46-notes (1.46.0-wmf.15; 2026-02-10).Mon, Feb 9, 10:00 AM
OKryva-WMF edited projects, added Product Safety and Integrity (Sprint Flower (Feb 9 - Feb 27)); removed Product Safety and Integrity (Sprint Daffodil (Jan 19 - Feb 6)).Tue, Feb 10, 8:28 AM
OKryva-WMF moved this task from Backlog to Needs QA on the Product Safety and Integrity (Sprint Flower (Feb 9 - Feb 27)) board.Tue, Feb 10, 8:29 AM
gerritbot added a comment.Tue, Feb 10, 10:11 PM

Change #1238436 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/ConfirmEdit@wmf/1.46.0-wmf.14] ConfirmEdit: Add error count threshold for apiUrl health checks

https://gerrit.wikimedia.org/r/1238436

gerritbot added a project: Patch-For-Review.Tue, Feb 10, 10:11 PM
gerritbot added a comment.Tue, Feb 10, 10:27 PM

Change #1238436 merged by jenkins-bot:

[mediawiki/extensions/ConfirmEdit@wmf/1.46.0-wmf.14] ConfirmEdit: Add error count threshold for apiUrl health checks

https://gerrit.wikimedia.org/r/1238436

Stashbot added a comment.Tue, Feb 10, 10:28 PM

Mentioned in SAL (#wikimedia-operations) [2026-02-10T22:28:06Z] <kharlan@deploy2002> Started scap sync-world: Backport for [[gerrit:1238436|ConfirmEdit: Add error count threshold for apiUrl health checks (T416817)]]

Maintenance_bot removed a project: Patch-For-Review.Tue, Feb 10, 10:30 PM
Stashbot added a comment.Tue, Feb 10, 10:32 PM

Mentioned in SAL (#wikimedia-operations) [2026-02-10T22:32:20Z] <kharlan@deploy2002> kharlan: Backport for [[gerrit:1238436|ConfirmEdit: Add error count threshold for apiUrl health checks (T416817)]] synced to the testservers (see https://wikitech.wikimedia.org/wiki/Mwdebug). Changes can now be verified there.

Stashbot added a comment.Tue, Feb 10, 10:40 PM

Mentioned in SAL (#wikimedia-operations) [2026-02-10T22:40:40Z] <kharlan@deploy2002> Finished scap sync-world: Backport for [[gerrit:1238436|ConfirmEdit: Add error count threshold for apiUrl health checks (T416817)]] (duration: 12m 34s)

ReleaseTaggerBot edited projects, added MW-1.46-notes (1.46.0-wmf.14; 2026-02-03); removed MW-1.46-notes (1.46.0-wmf.15; 2026-02-10).Tue, Feb 10, 11:00 PM
OKryva-WMF edited projects, added Product Safety and Integrity (Sprint Crocus (Mar 2 - Mar 20)); removed Product Safety and Integrity (Sprint Flower (Feb 9 - Feb 27)).Mon, Mar 2, 4:18 PM
OKryva-WMF moved this task from Backlog to Needs QA on the Product Safety and Integrity (Sprint Crocus (Mar 2 - Mar 20)) board.Mon, Mar 2, 4:19 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits