Page MenuHomePhabricator
Create Task
Maniphest T401673

Consider following redirects in mwbot
Closed, DeclinedPublic
Actions

Description

In T393428#11077617, thanks to additional logging introduced in T401653, we discovered that a lot of selenium-daily beta jobs were failing because the config used the wmflabs.org domain instead of the new wmcloud.org T289318. While the specific issue is being fixed, I think in general it would be nice if mwbot followed redirects by default.

I don't fully appreciate the implications of this change, but if this makes sense, then the feature request should be upstreamed. (Else this task can be closed as declined)

Related Objects
Search...

StatusSubtypeAssignedTask
 ResolvedPeterT405079 Stop using mwbot in Selenium tests
 DeclinedNoneT401673 Consider following redirects in mwbot

Event Timeline

Daimona created this task.Aug 12 2025, 12:05 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 12 2025, 12:05 PM
Peter mentioned this in T403748: Investigate what's needed to make core webdriver.io tests independent of mwbot to make sure core uses maintained code.Sep 5 2025, 8:55 AM
zeljkofilipin edited projects, added: Testing Support; removed: MediaWiki-Core-Tests.Sep 19 2025, 11:06 AM
zeljkofilipin added a parent task: T405079: Stop using mwbot in Selenium tests.Sep 19 2025, 11:12 AM
Peter subscribed.Oct 21 2025, 3:57 PM

Let me look at our new implementation if this is done automatically in wdio-mediawiki 6.0.0.

Peter mentioned this in T408087: Make it clear that the mwbot replacement do not follow redirects.Oct 23 2025, 10:05 AM
Peter added a comment.Oct 23 2025, 12:40 PM

We replaced mwbot with our own implementation in core using NodeJS built in fetch. Right now it tries to follow redirects but if I understand the spec correctly a redirect will change the POST to a GET and drop the data we are posting the body. I made a patch to disable redirects so that if you try use the API against something that redirects, it will get an error. I think that will make sense since like it is now, it will not work either, it better to get a failure fast.

Maybe we could build something that catches the redirect and re-POST but I think that is overkill right now. Lets verify that the new implementation works fine and see if there other things that needs to be tuned.

Peter closed this task as Declined.Oct 23 2025, 12:41 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