[13:12:39] [1/2] Does somebody know how to fix (`Sessions are disabled for load entry point`)? [13:12:40] [2/2] It seems to be caused by a message retrieval in , and I don't think we can get upstream to fix this because the extension is maintained by BlueSpice and they don't have a proper public issue tracker [14:42:59] Do they have a bug report process at all [15:03:00] Don't load something that requires session data over load.php [15:03:10] Load.php should be generic so it's a safeguard [15:03:22] If you get that error, it's a catch to prevent a security bug [19:47:20] Not sure; we're not a customer, so probably not [19:47:52] I could just email them since I've reported vulnerabilities to them before, but I feel like it would still take ages for them to fix it [19:48:08] Well yeah but how do I load the message without using session data [19:48:22] I'm not really interested in rewriting the logic [19:51:15] so parsed message and removeredlinks double whammy [19:52:46] hmm, have removeredlinks assume logged out if MW_NO_SESSION? [19:53:26] oh wait I missed that part of the backtrace [19:53:39] probably [19:55:34] i have told someone to disable removeredlinks because of that iirc [19:55:44] i swear i added something to a task too but i might be hallucinating [19:55:54] yeah I remember seeing that issue mentioned before [19:56:10] https://github.com/marohh/mediawikiRemoveRedlinks/issues/4 [19:56:37] Looks like they rewrote it and forgot about this issue [20:00:47] I've managed to reproduce it locally, I'll probably submit a pr [20:13:27] [20:16:48] I wonder if this is the bug from before then what was causing red links to dissapear for logged in users? [20:16:54] I had to disable that extension because of that [20:17:26] that was fixed [20:17:36] oh [20:18:28] https://issue-tracker.miraheze.org/T13894 [20:20:15] Anybody wanna review this so I can apply the patch locally? I think it might take a while for the maintainer to respond and review my PR [20:20:41] you'll probably have to do what marty did [20:21:01] will look now [20:21:02] the maintainer is watching the repo [20:21:05] they should've gotten an email already [20:21:05] not that i'll have anything to ad but [20:21:15] you'd think [20:21:28] https://cdn.discordapp.com/attachments/1006789349498699827/1448409299800952832/image.png?ex=693b27c7&is=6939d647&hm=ab678a2fa8a5fb6adc402b12d1993adca35a0a3ac4711d960d9b5e6e4a34cc64& [20:21:58] for all we know they might not even see them cause they just dgaf [20:22:15] LGTM [20:22:30] basically I dont think it's a coincidence they only merged it after marty emailed