Page MenuHomePhabricator

Finish 1.35 update (OAuth)
Closed, ResolvedPublic

Description

See also Github issue.

Why needed: Currently on WBstack, MW 1.34 is still running for OAuth internal usage. Having a second service running uses more resources and can be confusing, as there are 2 versions of MW deployed on the platform (one seen by the public & one for internal oauth API calls)

Impact: Making this improvement would reduce resource load on WBstack, which speeds up editing speed “for free”. This could also end up exposing additional bugs in MW 1.35, OAuth, Wikibase or Quickstatements that should be fixed for 3rd party users overall.

Tech notes:

Error that stopped the upgrade originally:

Fatal error: Uncaught Error: Call to undefined method stdClass::doAuthorizationRedirect() in /var/www/html/php/Widar.php:364 Stack trace: #0 /var/www/html/php/Widar.php(68): Widar->authorize() #1 /var/www/html/php/Widar.php(262): Widar->process_request('action') #2 /var/www/html/index.php(27): Widar->render_reponse(false) #3 {main} thrown in /var/www/html/php/Widar.php on line 364

Links of interest

Acceptance Criteria

  • MW 1.34 is no longer deployed on WBstack
  • All code for deploying code for MW 1.34 is gone
  • Widar and Quickstatements continue to function (i.e. you can log in)
  • If this is a fix elsewhere, it is upstreamed

Details

Other Assignee
toan

Event Timeline

Merged https://github.com/wbstack/mediawiki/pull/93 which should fix the issue
And updated prod quickstatements to point to the new 135 mediawiki image https://github.com/wbstack/deploy/commit/d4c26c2d8d84c5f3b1036ea7b252b35d9f24e8b1

And it all looks good!
Need to finish the deployment at some point thoug!

Addshore added a subscriber: toan.

I'll steal this and finish pushing the buttons