Hey all,

Something you might have more instantaneous insight on than I.

I deployed a fix to CentralNotice today [1] that should have started serving mobile URLs for CN to mobile clients. It does this by querying MobileContext::singleton()->shouldDisplayMobileView() inside of the ResourceLoaderGetConfigVars hook. However, it appears that either... that function is returning false, or... getMobileUrl is not properly mangling URLs.

I suspect it's that the function is returning false because:

* I deployed a 'fix' earlier that just used getMobileUrl() which redirected everyone to the mobile URLs. And...

* There's a blob of code that I added in this patch that will always mangle URLs and it seems like that's working just fine.

I've tested on my local machine with MF loaded first, and MF loaded last and can not reproduce the cluster behaviour. So... thoughts?

[1] https://gerrit.wikimedia.org/r/#/c/92806/

~Matt Walker