[08:28:20] 6Multimedia, 6Commons, 10MediaWiki-extensions-UploadWizard: Notify user of blocked IP before the upload process starts, not when it completes - https://phabricator.wikimedia.org/T105863#1453214 (10El_Grafo) 3NEW [11:00:12] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453464 (10Steinsplitter) @Gilles can this function then be enabled, please? [11:08:43] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453476 (10Yann) Hi, This has been open for more than 3 years. What's the current status? Thanks. [11:38:46] 6Multimedia, 6Commons, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review: Transferring PD-images from Flickr becomes 'undefined'-license - https://phabricator.wikimedia.org/T105629#1453534 (10zhuyifei1999) [11:38:48] 6Multimedia, 6Commons, 10MediaWiki-extensions-UploadWizard, 7Tracking: UploadWizard Flickr (tracking) - https://phabricator.wikimedia.org/T45450#1453533 (10zhuyifei1999) [12:07:42] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453613 (10Reedy) >>! In T35186#1367005, @Rillke wrote: > @gilles can you also run > ``` > which avconv > which ffmpeg > ``` > > Would video rotation a... [12:19:07] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453625 (10Reedy) >>! In T35186#1453476, @Yann wrote: > Hi, This has been open for more than 3 years. What's the current status? Thanks. Open time does... [12:54:33] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453674 (10Steinsplitter) >>! In T35186#1453625, @Reedy wrote: > I guess it needs testing both non WMF cluster that the code actually works... And if so... [12:55:56] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453678 (10Reedy) >>! In T35186#1453674, @Steinsplitter wrote: >>>! In T35186#1453625, @Reedy wrote: >> I guess it needs testing both non WMF cluster th... [13:07:33] (03CR) 10Zhuyifei1999: [WIP] Add flickr checking to backend (032 comments) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/212948 (https://phabricator.wikimedia.org/T100062) (owner: 10Brian Wolff) [13:35:48] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453783 (10Reedy) But we still have the "routing" issues to get the image scalers to do the scaling... [13:46:33] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453811 (10Anomie) I don't see a whole lot of point to enabling the existing code for testing on the cluster, since we're not going to be able to use th... [14:04:57] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453848 (10Steinsplitter) >>! In T35186#1453811, @Anomie wrote: > I don't see a whole lot of point to enabling the existing code for testing on the clus... [14:09:45] 6Multimedia, 6Commons, 10MediaWiki-API, 10MediaWiki-Uploading: Provide a well-performing API to rotate an image - https://phabricator.wikimedia.org/T35186#1453853 (10Anomie) Sure, I'll do that in my [[https://en.wiktionary.org/wiki/copious_free_time|copious free time]]. Seriously though, it's on my list a... [14:29:16] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 10VisualEditor, 10VisualEditor-MediaWiki: MediaViewer does not work directly after saving a VisualEditor edit - https://phabricator.wikimedia.org/T101951#1453921 (10Krenair) I don't think there is a VE-specific hook... We do fire wikipage.content. I thi... [17:01:38] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 7Tracking: Chunked upload (tracking) - https://phabricator.wikimedia.org/T61914#1454525 (10JeanFred) [17:01:41] 6Multimedia, 6Commons, 10MediaWiki-Uploading: Chunked upload file fails to reconstitute... - https://phabricator.wikimedia.org/T105791#1454524 (10JeanFred) [17:01:53] 6Multimedia, 6Commons, 10MediaWiki-Uploading: Chunked upload file fails to reconstitute... - https://phabricator.wikimedia.org/T105791#1451693 (10JeanFred) [17:01:56] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 7Tracking: Chunked upload (tracking) - https://phabricator.wikimedia.org/T61914#637106 (10JeanFred) [17:02:03] 6Multimedia, 6Commons, 10MediaWiki-Uploading: Chunked upload file fails to reconstitute... - https://phabricator.wikimedia.org/T105791#1451693 (10JeanFred) [17:02:05] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 7Tracking: Chunked upload (tracking) - https://phabricator.wikimedia.org/T61914#1454530 (10JeanFred) [18:40:11] tgr: What was The Trick™ for making CORS requests work? [18:42:13] Found it, https://www.mediawiki.org/wiki/API:Cross-site_requests [18:43:26] &origin=originhostname :) [18:45:16] thedj: I found it!!!! [18:45:31] i asked csteip to check if we actually need that... btw [18:45:53] basically if the html spec author says we are doing it wrong... i tend to put some faith in that. [18:46:27] we needed it before, because we had implemented it incorrectly. but now... i think it's just an unnecessary complication. [18:46:36] https://phabricator.wikimedia.org/T62835 [18:47:43] lemme tag that with the security reviews project. [18:49:20] Thanks. [18:49:30] I'm happy to add some lines adding a default parameter for that [18:49:32] But whatever [18:51:41] Bloody hell, is CORS not enabled on beta? [18:52:57] oh quite possibly yes. [18:54:18] It looked like it was last time I checked but I guess not [18:54:36] I should find Andre. [18:56:22] update that ticket in hopes that chris can verify that we are being overly cautious here, beyond reasonability :) [18:56:28] updated [18:59:39] thedj: I set defaultOptions.parameters.origin = document.location.host; is that not enough? [18:59:46] Because it's still not going. [19:00:50] See in particular http://en.wikipedia.beta.wmflabs.org/wiki/User:MarkTraceur/commonsUpload.js [19:01:57] The request has the right parameter I think [19:02:43] Maybe this is because I don't know about CentralAuth and it can't get a token  [19:04:13] Do I need to do, like, action=centralauthtoken and then use that for the upload? [19:04:15] marktraceur: you need the origin on the url of the api, not on the data . [19:04:33] w/api.php?origin=commons.wikimedia.org [19:04:41] thedj: Yes, that's what I'm doing, it's a GET request for the token [19:05:08] You mean origin=en.wikipedia.org surely, because that's where I'm starting and it's requesting data from Commons [19:05:51] yeah wherever you are starting of course. [19:06:01] OK. [19:06:17] Maybe I should corner anomie, then. [19:06:37] ehm, is beta.wmflabs.org in the commons production whitelist though [19:07:50] can you just paste me the raw XHR request from the inspector onto a pastebin or something ? [19:08:11] so that i can see what headers and body etc is being passed ? [19:09:36] oh wait, are you doing something that requires credentials ? [19:09:46] because they don't automatically get send of course. [19:47:22] 6Multimedia, 10MediaWiki-File-management, 6Performance-Team, 5Patch-For-Review: Auto-cleanup least accessed thumbnails option for local file storage - https://phabricator.wikimedia.org/T105786#1454986 (10Gilles) I've been thinking about this some more, from the perspective of the least amount of work to ge... [20:39:15] MatmaRex: You available to merge a fix for mw.Api.upload so I don't need to keep hacking it to death? [20:42:39] James_F: Want to merge a patch? :D [20:54:16] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Create video/audio transcode service to replace shell-outs to avconv/ffmpeg2theora - https://phabricator.wikimedia.org/T105951#1455210 (10brion) 3NEW [20:55:05] Never mind, anomie is awesome [20:59:49] marktraceur: ah, you needed to cross origin login .. wow. :) [21:08:13] thedj: Yeah this is a pain in the ass [21:09:37] that's why we need proper api's in core to handle the pain. you are doing good work here. :) [21:10:01] Yup. [21:10:06] you will get the ultimate satisfaction at some point of not a single user thanking you because stuff finally works, so they won't complain :) [21:10:12] Heh [21:11:50] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Support transcoding of H.264+PCM Matroska (.mkv) source files with OpenH264 - https://phabricator.wikimedia.org/T105957#1455290 (10brion) 3NEW [21:55:02] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 5Patch-For-Review, 5WMF-deploy-2015-07-07_(1.26wmf13): MediaViewer cannot be closed in Microsoft Edge - https://phabricator.wikimedia.org/T104381#1455426 (10Dstoreymsft) A fix for this issue is in MS Edge in the latest public build of Windows 10, build... [22:08:30] marktraceur: i am now. i also have a upload api patch for you to merge [22:18:54] 6Multimedia, 10MediaWiki-File-management, 6Performance-Team, 5Patch-For-Review: Auto-cleanup least accessed thumbnails option for local file storage - https://phabricator.wikimedia.org/T105786#1455483 (10Gilles) FileBackendMultiWrite seems to provide the basic functionality I need, although I will probably... [22:18:54] marktraceur: Sure? [22:24:56] Never mind, guys [22:25:01] OK. [22:25:28] MatmaRex: Link me, I'll do it after dinner [22:25:41] marktraceur: https://gerrit.wikimedia.org/r/224451 [22:25:57] bah, doesn't merge already?… [22:26:24] * MatmaRex rebases [22:35:37] 6Multimedia, 6Commons, 10MediaWiki-Uploading: Chunked upload file fails to reconstitute... - https://phabricator.wikimedia.org/T105791#1455533 (10Tgr) Reproduced on test2wiki, there are a bunch of errors like this: ``` 2015-07-15 22:00:51 mw1004 test2wiki error-json INFO: {"id":"43b73c70","type":"ErrorExcept... [22:36:49] 6Multimedia, 6Commons, 10MediaWiki-Uploading: Chunked upload file fails to reconstitute... - https://phabricator.wikimedia.org/T105791#1455534 (10Tgr) ...which is apparently just some crappy hack in `wfShellExec()` which clogs up the error logs. Duh. [22:42:23] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Support MPEG DASH for multiple webm files - https://phabricator.wikimedia.org/T44591#1455599 (10brion) Update ping: ogv.js is now able to stream a large file via multiple Range: requests, and will be able to do adaptive streaming with them in the future,... [22:52:17] 6Multimedia, 6Commons, 10MediaWiki-Uploading: Chunked upload file fails to reconstitute... - https://phabricator.wikimedia.org/T105791#1455608 (10Tgr) A bit hardto read the error logs since enormous chunks of the DJVU file are included, but these show up often: ``` 2015-07-15 22:00:11 mw1203 test2wiki wfDebu... [22:55:51] (03PS1) 10Brion VIBBER: Revert no-longer-needed fix for MS Edge browser [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/224969 (https://phabricator.wikimedia.org/T104381) [22:57:43] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 5Patch-For-Review, 5WMF-deploy-2015-07-07_(1.26wmf13): MediaViewer cannot be closed in Microsoft Edge - https://phabricator.wikimedia.org/T104381#1455639 (10brion) @Dstoreymsft fix confirmed in Windows 10 build 10240 -- thanks! Revert of the workaroun... [23:00:39] (03CR) 10Gergő Tisza: "How is MS Edge updated? Is it a Chrome-like autoupdate model where we can be reasonably sure that after a few weeks of release no one uses" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/224969 (https://phabricator.wikimedia.org/T104381) (owner: 10Brion VIBBER) [23:04:05] (03CR) 10Brion VIBBER: "For the meantime, all Windows 10 users are on prerelease builds that are *very* aggressively updated, and the fix has made it in before th" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/224969 (https://phabricator.wikimedia.org/T104381) (owner: 10Brion VIBBER) [23:10:08] 6Multimedia, 6Commons, 10MediaWiki-extensions-MultimediaViewer, 5Patch-For-Review, 7user-notice: More restriction icons! - https://phabricator.wikimedia.org/T102693#1455697 (10Tgr) [23:12:07] (03CR) 10Gergő Tisza: "So this is good to go?" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/224969 (https://phabricator.wikimedia.org/T104381) (owner: 10Brion VIBBER) [23:12:23] (03CR) 10Brion VIBBER: "Yep!" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/224969 (https://phabricator.wikimedia.org/T104381) (owner: 10Brion VIBBER) [23:18:12] (03CR) 10Gergő Tisza: [C: 032] "Cool, thanks for keeping an eye on this!" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/224969 (https://phabricator.wikimedia.org/T104381) (owner: 10Brion VIBBER) [23:19:13] (03Merged) 10jenkins-bot: Revert no-longer-needed fix for MS Edge browser [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/224969 (https://phabricator.wikimedia.org/T104381) (owner: 10Brion VIBBER) [23:23:31] (03CR) 10Bartosz Dziewoński: [C: 032] Reduce number of RL modules in MultimediaViewer [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/220621 (https://phabricator.wikimedia.org/T103706) (owner: 10Gilles) [23:23:57] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 6Performance-Team: Reduce number of RL modules in MultimediaViewer - https://phabricator.wikimedia.org/T103706#1455735 (10matmarex) 5Open>3Resolved [23:24:29] (03Merged) 10jenkins-bot: Reduce number of RL modules in MultimediaViewer [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/220621 (https://phabricator.wikimedia.org/T103706) (owner: 10Gilles) [23:24:49] 6Multimedia, 6Commons, 10Wikimedia-Media-storage, 7Tracking: Thumbnail/imagescaler (tracking) - https://phabricator.wikimedia.org/T43371#1455749 (10Aklapper)