[00:17:14] tgr: if you are bored -- T147992 [00:17:15] T147992: https://commons.wikimedia.org/wiki/File:Map_of_Hindoostan,_1788,_by_Rennell.jpg can't create thumbs - https://phabricator.wikimedia.org/T147992 [00:45:30] bd808: eh, poked at it to see how hard it would be to get sane logging (the exact shell command that failed, at least) [00:46:01] that would be nicely helpful :) [00:46:30] it would require rewriting most of the mediahandler hierarchy to use StatusValue instead of boolean/int error flags, or a log context passed in to every function or something [00:46:38] decided I am not that bored [00:48:47] maybe something very hacky could be done on top of T142313, like always storing the last shell command [00:48:48] T142313: Add global logging context - https://phabricator.wikimedia.org/T142313 [13:32:09] legoktm, bd808: My first thought is "how complicated are these lint checks?" My second thought is that you might be able to avoid deletes if you key on rev_id rather than page_id, unless the linting is happening after template expansion, although then returning a list of broken *pages* would be more complicated because you'd have to post-filter by whether the revision is the latest. [17:02:25] * anomie is not hugely fond of an API endpoint that's so internal that it needs IP-based restrictions [17:06:10] other suggestions on how to move the data from parsoid to MW? :) [17:08:20] we could create a special userright and system user and auth over OAuth, but I don't think that's any better and adds more complexity imo [17:09:47] Just because I can't think of a better way doesn't mean I have to like it. [17:10:42] heh ok :P [17:17:53] greg-g: Ha, I submitted a patch for T136401 3 minutes before you emailed about it! How's that for response time? ;) [17:17:54] T136401: Exception "Invalid marker...Expansion depth limit exceeded" thrown when template expansion limit is exceeded inside a strip marker - https://phabricator.wikimedia.org/T136401 [17:19:38] anomie: epic! [17:19:54] anomie: can I borrow your time machine? [17:23:18] greg-g: Heh. Really I just saw the new activity and the fact that someone made it a release blocker, and decided to just fix it already. [17:23:29] thanks :) [17:55:13] anomie: btw, you want to reply with a "beat ya!" or me? :) [17:55:31] greg-g: You can reply [17:55:37] kk [18:50:11] anomie: I submitted https://gerrit.wikimedia.org/r/315722 to put EditFilterMergedContent errors in an errorbox