Page MenuHomePhabricator

Enable push subscription cleanup on the Beta Cluster
Closed, DuplicatePublic

Description

  • Create a subscription manager user on Beta Metawiki
  • Update push subscription service configuration with subscription manager user credentials and API config (NOTE: Don't expose the subscription manager password publicly via the service configuration in instance hiera settings!)
  • Confirm that bad subscriptions are deleted on message request failure

Event Timeline

Mholloway renamed this task from Enable subscription cleanup on the Beta Cluster to Enable push subscription cleanup on the Beta Cluster.Sep 1 2020, 7:28 PM
Mholloway raised the priority of this task from Medium to Needs Triage.Sep 2 2020, 3:34 PM
Mholloway triaged this task as Medium priority.
Mholloway updated the task description. (Show Details)

Today's progress on this: After improving MW API error handling I was able to verify that API requests in Beta that require CSRF tokens are failing with badtoken errors, and after adding HTTP request debug logging I was able to verify that this is because session cookies are not being set on requests to the MediaWiki API as expected. The reason for this is that in both Beta and production MediaWiki is currently configured such that the Secure attribute is unconditionally set on all cookies we receive from MediaWiki, and we are making internal connections to MediaWiki via the insecure http: protocol (as TLS is AFAIK unavailable for Beta Cluster-internal requests). Cookies with the Secure attribute are only set on requests over secure (https:) connections.

I think we may have better authentication options than action=clientlogin, which is what we're currently using. On Monday I'll try coding up a bot password login method and see where that gets us.