[02:27:35] 6Multimedia, 6operations: Add monitoring of upload rate on commons to icinga alerts - https://phabricator.wikimedia.org/T92322#1419551 (10Tgr) https://gerrit.wikimedia.org/r/#/c/222224/ will make this easy again. [02:43:19] 6Multimedia, 10MediaWiki-File-management: Commons images with embedded color profiles shown incorrectly - https://phabricator.wikimedia.org/T44827#1419577 (10Bawolff) >>! In T44827#1418928, @Qgil wrote: >>>! In T44827#446700, @McZusatz wrote: >> The current versions of all of those files were reencoded and reu... [02:47:08] 6Multimedia, 10MediaWiki-File-management: Commons images with embedded color profiles shown incorrectly - https://phabricator.wikimedia.org/T44827#1419587 (10Bawolff) >>! In T44827#1418928, @Qgil wrote: >>>! In T44827#446700, @McZusatz wrote: >> The current versions of all of those files were reencoded and reu... [02:57:07] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: MediaViewer cannot be closed in Microsoft Edge - https://phabricator.wikimedia.org/T104381#1419617 (10Tgr) Finding out how many people use Edge is T104531. [03:30:23] 6Multimedia, 10MediaWiki-File-management: Commons images with embedded color profiles shown incorrectly - https://phabricator.wikimedia.org/T44827#1419693 (10Tgr) > Although if we did discover there is some specific type of profile that didn't work with web browsers, we could perhaps detect and strip, although... [05:51:18] 6Multimedia, 10MediaWiki-File-management: Commons images with embedded color profiles shown incorrectly - https://phabricator.wikimedia.org/T44827#1419810 (10Qgil) (Just to be clear, I don't have an opinion about this bug. It sounds like a bit of a corner case?) [06:09:08] 6Multimedia, 10MediaWiki-File-management: Commons images with embedded color profiles shown incorrectly - https://phabricator.wikimedia.org/T44827#1419814 (10Bawolff) I did some investigating of the file. I think firefox, and chrome are in the wrong (I believe both use qcms to handle colour profiles, although... [06:26:07] 6Multimedia, 10MediaWiki-General-or-Unknown, 7HTTPS, 5Patch-For-Review: PhpHttpRequest should not check host against CN x509 attribute - https://phabricator.wikimedia.org/T75199#1419826 (10Bawolff) > I tested peer_name with PHP 5.5.6 and it works correctly Did you check that it correctly validates certifi... [06:35:47] 6Multimedia, 10MediaWiki-File-management: Commons images with embedded color profiles shown incorrectly - https://phabricator.wikimedia.org/T44827#1419829 (10Bawolff) Hmm, I was looking at a really old version of qcms, which was part of my confusion [07:05:48] 6Multimedia, 10MediaWiki-File-management: Commons images with embedded color profiles shown incorrectly - https://phabricator.wikimedia.org/T44827#1419858 (10Tgr) So... # can we upstream this? # is there a practical way to detect on upload whether a file is affected? # is there a practical way to convert affec... [07:07:02] tgr: I'm writing an upstream bug now [07:08:33] thanks! [07:08:46] * tgr has no clue about color profiles [07:10:56] I don't really have that great an understanding either [07:11:56] IMO UW at some point needs a framework of suggesting pre-save transformations on uploads to the user (mostly for converting to free formats, as format problems seem to be the main reason for aborted uploads), and a color profile transformation would fit into that nicely [07:12:51] I'd actually rather it be transformed as part of the thumbnailing, so we always have the original in case colour profile transform explodes (Which seems to be the thing color profiles are mostly good for) [07:15:54] that would 1) require thumbnailing even for full sizes and 2) would be still hard on downloaders [07:16:11] 1) is something we need to tackle eventually anyway [07:16:55] I'm not sure what could be done about 2) [07:18:22] 1) we already do if there's an exif rotation tag, so it wouldn't be that much of a change [07:18:37] 2) is more the fuzzy ux problem that's always hard :s [07:19:19] that's true, just need to set the needtransform flag or what's it called [07:19:56] I wonder if at some point it would make sense to generate a separate "thumbnail" for download purposes [07:20:38] there are some transforms that would make sense for download, like putting the author/license into the metadata [07:21:49] although I suspect 99.9% of the people download via right click-save as on the thumbnail anyway [07:23:11] 6Multimedia, 10MediaWiki-File-management, 7Upstream: Commons images with embedded color profiles shown incorrectly - https://phabricator.wikimedia.org/T44827#1419903 (10Bawolff) [07:24:30] 6Multimedia, 10MediaWiki-File-management, 7Upstream: Commons images with embedded color profiles shown incorrectly - https://phabricator.wikimedia.org/T44827#446344 (10Bawolff) Filed upstream bug at https://bugzilla.mozilla.org/show_bug.cgi?id=1179671 [Of course, even if that gets fixed, that doesn't help u... [07:24:59] yes, putting metadata in the files would be nice [07:25:08] I'd be happy even if we just did that for thumbs [07:31:09] 6Multimedia, 10MediaWiki-General-or-Unknown, 7HTTPS, 5Patch-For-Review: PhpHttpRequest should not check host against CN x509 attribute - https://phabricator.wikimedia.org/T75199#1419923 (10Seb35) I aggree php-curl is really better than PHP on this case, but, on Debian at least, it is not installed by defau... [09:29:49] 6Multimedia, 6Commons, 10MediaWiki-Uploading: intermittent error on chunk upload of tiff file - https://phabricator.wikimedia.org/T73256#1420136 (10jeblad) Is this done, or is there still some work left? [11:25:39] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: MediaViewer cannot be closed in Microsoft Edge - https://phabricator.wikimedia.org/T104381#1420397 (10brion) I get a JS error when I test it in Edge in the current Windows 10 build: ``` Unspecified error. ``` in this line: ``` this.browserHistory.pushSta... [11:50:24] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: MediaViewer cannot be closed in Microsoft Edge - https://phabricator.wikimedia.org/T104381#1420445 (10brion) Isolated test case to report to Microsoft: https://brionv.com/misc/edge-pushstate/ [11:51:07] (03PS1) 10Brion VIBBER: Workaround apparent bug in Edge's history.pushState [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/222279 (https://phabricator.wikimedia.org/T104381) [11:52:07] (03CR) 10jenkins-bot: [V: 04-1] Workaround apparent bug in Edge's history.pushState [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/222279 (https://phabricator.wikimedia.org/T104381) (owner: 10Brion VIBBER) [13:54:47] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Spurious Ogg Vorbis transcodes of WebM videos - https://phabricator.wikimedia.org/T104571#1420911 (10brion) 3NEW a:3brion [13:55:23] (03PS1) 10Brion VIBBER: Tiny bug in error condition [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/222294 [14:31:10] (03PS7) 10Sn1per: Moar restriction icons! [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/219534 (https://phabricator.wikimedia.org/T102693) [15:18:58] tgr|away: Are you or gilles still using ImageMetrics data by any chance? [15:34:56] marktraceur: Meeting? [15:35:32] Oh right. [15:35:36] One moment.... [15:35:39] :-) [15:35:59] Needed to stand up to turn off the AC. You'll see why that took time in a second. [15:36:12] Because hot? [15:36:16] I can cope with the noise… [16:48:22] * marktraceur has fixed and rebased all the things. [17:00:00] marktraceur: I wanted to do a last pass on the CORS data before killing it [17:00:13] OK, we'll leave IM data up then [17:00:32] I figured the image performance data can just be left forever, it's nice to have perf dashboards [17:02:07] 6Multimedia, 10MediaWiki-Uploading, 7JavaScript: Add a model for uploads-in-process in MediaWiki core - https://phabricator.wikimedia.org/T103413#1421658 (10MarkTraceur) a:3MarkTraceur [17:25:20] tgr: Logging data slows things down; if no-one's pay attention… [17:26:24] you need to log data to know whether you have slowed down in the first place though [17:26:45] I don't think it's a good idea to have any major product without performance data [20:15:12] 6Multimedia, 6Commons, 10MediaWiki-extensions-UploadWizard, 5Patch-For-Review, and 2 others: Remove Flickr checking from client side - https://phabricator.wikimedia.org/T100062#1422572 (10gpaumier) [22:58:38] 6Multimedia, 10MediaWiki-General-or-Unknown, 7HTTPS, 5Patch-For-Review: PhpHttpRequest should not check host against CN x509 attribute - https://phabricator.wikimedia.org/T75199#1423358 (10Seb35) For completeness here (I spoke about it in the gerrit patch), I was wrong about the resolution for PHP 5.5.6: p... [23:37:41] 6Multimedia, 6Commons, 10MediaWiki-Database, 7Database, and 3 others: internal_api_error_DBQueryError: Database query error while (mass) deleting file over api - https://phabricator.wikimedia.org/T98706#1423438 (10aaron) 5Open>3Resolved