[07:12:33] !log Deployment-prep Applied SQL queries per [[phab:T231058#5433197]] [07:12:36] rxy: Unknown project "Deployment-prep" [07:12:37] T231058: No merged changes are deployed on betacluster - https://phabricator.wikimedia.org/T231058 [07:12:46] !log deployment-prep Applied SQL queries per [[phab:T231058#5433197]] [07:12:50] Logged the message at https://wikitech.wikimedia.org/wiki/Nova_Resource:Deployment-prep/SAL [08:35:29] !log tools.speed-patrolling repurposing this defunct tool (T212968) for some ad-hoc CORS testing [08:35:32] Logged the message at https://wikitech.wikimedia.org/wiki/Nova_Resource:Tools.speed-patrolling/SAL [08:41:47] !help I sent a pull request to the pagepile tool to add Access-Control-Allow-Origin: * headers to some responses, but the headers aren’t being sent [08:41:47] lucaswerkmeister: If you don't get a response in 15-30 minutes, please create a phabricator task -- https://phabricator.wikimedia.org/maniphest/task/edit/form/1/?projects=wmcs-team [08:42:19] the code seems to be deployed (in ~tools.pagepile/public_html/api.php), and when I try it locally the header() call works [08:42:48] so I thought perhaps the Cloud proxy was filtering out that header, but I tried it in other tools, including python3.5, php5.6 and php7.2 backends (all k8s) and it works [08:42:59] so now I’m out of ideas why the header is getting lost. does anyone else have any? [08:48:16] gridengine lighttpd doesn’t seem to filter it out either [08:53:01] I don’t think php webservices need to be restarted for changes to become effective, either (just tried that out) [09:03:15] lucaswerkmeister: Can you make a task for that? I want to help take a look, but folks are traveling at the moment. [09:04:00] Ah...apparently my irc nick needs love...this is bstorm_ [09:04:35] sure, thanks :) [09:04:50] (though tbh in the meantime I’ll probably add aCORS-enabled “proxy” to one of my own tools, just to unblock myself ^^ [09:04:55] ) [09:18:24] Guest44962 / bstorm: nevermind, it’s an error in the PHP [09:18:37] misunderstood the control flow [09:21:02] yup, now it works, I’ll send another PR to magnus [09:21:04] sorry for the noise [09:37:12] here’s the fix if anyone’s curious https://bitbucket.org/magnusmanske/pagepile/pull-requests/7/fix-access-control-allow-origin-for/diff [19:17:09] !log tools.admin Update to f742092 Update text on error pages (T231090) [19:17:12] Logged the message at https://wikitech.wikimedia.org/wiki/Nova_Resource:Tools.admin/SAL [19:17:13] T231090: Error on Toolforge 404 (desktop site) - References non-existent sidebar - https://phabricator.wikimedia.org/T231090 [20:52:04] !log tools.wikibugs Updated channels.yaml to: ff7f5fed938e91a41c91200c105f00dc6b3142f1 Reading-Infrastructure-Team Phab boards have new names [20:52:06] Logged the message at https://wikitech.wikimedia.org/wiki/Nova_Resource:Tools.wikibugs/SAL