[{reqId}] {exception_url} BadMethodCallException: MediaWiki\Session\SessionProvider::preventSessionsForUser must be implemented when canChangeUser() is falseMaybe it's from NetworkSessions?
__METHOD__ resolving to the parent class is a bit unhelpful here.
Change #1194218 had a related patch set uploaded (by Gergő Tisza; author: Gergő Tisza):
[mediawiki/core@master] session: Log actual class name in preventSessionsForUser exception
Not related to the event platform.
Not really related to Babel either but it did break BabelAutoCreate::create() so adding for awareness.
I guess we should have a structure test that calls preventSessionsForUser() for every session provider.
BabelAutoCreate failing is not a big deal; a large number of wikis have blocked the account entirely and hence have their own processes for creating babel categories.
Change #1194218 merged by jenkins-bot:
[mediawiki/core@master] session: Log actual class name in preventSessionsForUser exception
Change #1194261 had a related patch set uploaded (by Gergő Tisza; author: Gergő Tisza):
[mediawiki/core@wmf/1.45.0-wmf.21] session: Log actual class name in preventSessionsForUser exception
Change #1194262 had a related patch set uploaded (by Gergő Tisza; author: Gergő Tisza):
[mediawiki/core@wmf/1.45.0-wmf.22] session: Log actual class name in preventSessionsForUser exception
Change #1194261 merged by jenkins-bot:
[mediawiki/core@wmf/1.45.0-wmf.21] session: Log actual class name in preventSessionsForUser exception
Change #1194262 merged by jenkins-bot:
[mediawiki/core@wmf/1.45.0-wmf.22] session: Log actual class name in preventSessionsForUser exception
Mentioned in SAL (#wikimedia-operations) [2025-10-07T21:30:44Z] <tgr@deploy2002> Started scap sync-world: Backport for [[gerrit:1194261|session: Log actual class name in preventSessionsForUser exception (T406566)]], [[gerrit:1194262|session: Log actual class name in preventSessionsForUser exception (T406566)]], [[gerrit:1194281|session: Log cache write flags in SessionStore::set() (T405633 T405634)]], [[gerrit:1194282|session: Log cache write flags in SessionStore::set() (T4056
Mentioned in SAL (#wikimedia-operations) [2025-10-07T21:34:47Z] <tgr@deploy2002> tgr: Backport for [[gerrit:1194261|session: Log actual class name in preventSessionsForUser exception (T406566)]], [[gerrit:1194262|session: Log actual class name in preventSessionsForUser exception (T406566)]], [[gerrit:1194281|session: Log cache write flags in SessionStore::set() (T405633 T405634)]], [[gerrit:1194282|session: Log cache write flags in SessionStore::set() (T405633 T405634)]] synced
Mentioned in SAL (#wikimedia-operations) [2025-10-07T21:40:21Z] <tgr@deploy2002> Finished scap sync-world: Backport for [[gerrit:1194261|session: Log actual class name in preventSessionsForUser exception (T406566)]], [[gerrit:1194262|session: Log actual class name in preventSessionsForUser exception (T406566)]], [[gerrit:1194281|session: Log cache write flags in SessionStore::set() (T405633 T405634)]], [[gerrit:1194282|session: Log cache write flags in SessionStore::set() (T405
Looking as far back as 12 months ago, it seems like we only had these spikes from Sept 22 - Oct 13, and it stopped since then, see: logstash query.
Do we still need to do anything here? Not much activity on https://phabricator.wikimedia.org/tag/networksession/, but spotted a potentially related task - T373826
Logstash only retains data for 3 months (so mid-August). Anyway these errors happen when newSystemUser() is called, which is rare. But they are also trivial to fix - NetworkSession just needs to implement a noop preventSessionsForUser() method (since it's for system-ish users in the first place, it doesn't need to worry about preventing normal users from loggin into system user accounts).
Change #1206384 had a related patch set uploaded (by D3r1ck01; author: Derick Alangi):
[mediawiki/extensions/NetworkSession@master] NetworkSessionProvider: Implement no-op preventSessionsForUser()
Change #1206384 merged by jenkins-bot:
[mediawiki/extensions/NetworkSession@master] NetworkSessionProvider: Implement no-op preventSessionsForUser()