[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-07-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

Krinkle krinklem...@gmail.com changed:

   What|Removed |Added

 CC||krinklem...@gmail.com
 Blocks||28425

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-14 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

Roan Kattouw roan.katt...@gmail.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #12 from Roan Kattouw roan.katt...@gmail.com 2011-01-14 19:36:19 
UTC ---
(In reply to comment #11)
 (In reply to comment #5)
  This should be fixed by r79835.
 
 On my wiki, r79835 indeed fixes the issue I saw
Marking as FIXED then.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-09 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

--- Comment #11 from Thomas Bleher thomasble...@gmx.de 2011-01-09 11:23:27 
UTC ---
(In reply to comment #5)
 This should be fixed by r79835.

On my wiki, r79835 indeed fixes the issue I saw, and I can upload SVG files
again.
I also looked at the code, and think that your description is correct.
Thanks for your work (the same goes to NealK)!

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

--- Comment #7 from Bryan Tong Minh bryan.tongm...@gmail.com 2011-01-08 
11:55:13 UTC ---
I guess we should make the extension optional, as it also breaks uploading
files without extension.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

--- Comment #8 from Neil Kandalgaonkar ne...@wikimedia.org 2011-01-08 
14:02:15 UTC ---
Well, to my knowledge, it was *always* an error to upload files without an
extension, at least for Wikimedia wikis -- an error is shown client-side.
That's why I felt confident making that a requirement. I was trying to change
as few things as possible.

I agree that it is a bit silly, since we are doing a mime type check anyway. If
I were designing the system from scratch, I would analyze the file server side
and canonicalize to one extension (rather than have, for instance, jpeg and jpg
co-existing). But the rule was just -- if there's no jpg, png, gift, etc, we
don't even let you begin to upload this.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

--- Comment #9 from Roan Kattouw roan.katt...@gmail.com 2011-01-08 15:24:06 
UTC ---
(In reply to comment #8)
 Well, to my knowledge, it was *always* an error to upload files without an
 extension, at least for Wikimedia wikis -- an error is shown client-side.
 That's why I felt confident making that a requirement. I was trying to change
 as few things as possible.
 
If the *target* (on-wiki) name is extensionless, maybe. But with stashed
uploading, the user hasn't even entered the target name yet, so I'm guessing
this is based on the local filename instead, which is evil: we should make
*zero* assumptions from the local filename, other than maybe prefilling the
target name.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

--- Comment #10 from Neil Kandalgaonkar ne...@wikimedia.org 2011-01-08 
22:15:50 UTC ---
(In reply to comment #9)

 We should make
 *zero* assumptions from the local filename, other than maybe prefilling the
 target name.

Agreed. On the other hand, if I spent time correcting everything about
MediaWiki that didn't make sense to me I'd never get anything done. ;) Still,
maybe we can fix THIS, since it seems to be actually harmful.

I'm sorry I haven't gotten around to addressing this yet.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

Thomas Bleher thomasble...@gmx.de changed:

   What|Removed |Added

 Blocks||26611

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

Bryan Tong Minh bryan.tongm...@gmail.com changed:

   What|Removed |Added

 AssignedTo|ne...@wikimedia.org |bryan.tongm...@gmail.com

--- Comment #4 from Bryan Tong Minh bryan.tongm...@gmail.com 2011-01-07 
19:09:13 UTC ---
For some reason your MediaWiki is not able to find mime.types, which in itself
is odd, but the fallback is broken. I'm looking into the proper format for it.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

Bryan Tong Minh bryan.tongm...@gmail.com changed:

   What|Removed |Added

 AssignedTo|bryan.tongm...@gmail.com|ne...@wikimedia.org

--- Comment #5 from Bryan Tong Minh bryan.tongm...@gmail.com 2011-01-07 
19:44:00 UTC ---
This should be fixed by r79835. However I think the real reason is UploadStash
requiring the stash key to have an extension. Is there a specific reason for
this? It seems to just work even if the extension does not match the mime type
at all.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2011-01-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

--- Comment #6 from Neil Kandalgaonkar ne...@wikimedia.org 2011-01-08 
05:25:09 UTC ---
There is no reason for the stash key to have an extension, other than paranoia
that some browsers might get confused when displaying previews or the full
thing. In fact, it used to not have an extension (or rather, it claimed to have
one and then discarded it). For simplicity I just made it have an extension.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2010-12-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

Roan Kattouw roan.katt...@gmail.com changed:

   What|Removed |Added

 CC||roan.katt...@gmail.com
 AssignedTo|wikibug...@lists.wikimedia. |ne...@wikimedia.org
   |org |

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2010-12-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

--- Comment #3 from Neil Kandalgaonkar ne...@wikimedia.org 2010-12-21 
01:02:22 UTC ---
Great work Thomas -- however it seems that other lines in the file follow the
same convention. Perhaps the intention is to create equivalency between a list
of mimetypes and a list of extensions, and one is supposed to just intuit that
a mimetype has a slash and an extension doesn't.

This might be a bug in MimeMagic -- if you want to look into it further, that
would be awesome, but I'm going to look into it anyway if you don't have the
time.



 (In reply to comment #1)
 I tracked the problem down a bit, and it seems to be a problem in
 includes/MimeMagic.php.
 
 MimeMagic has a list of predefined mime types, with assorted file extensions.
 The code that parses the types treats everything up to the first space as the
 mime type, everything behind that as a file extension. As you can see from the
 diff below, the svg line does not follow this convention, leading to an
 extension of image/svg.
 This blows up in UploadStash.
 
 With the change below, I can upload svg files again. But I'm not sure what the
 correct fix is here, as this particular line is in the code since r21411, from
 April 2007.
 
 --- a/includes/MimeMagic.php
 +++ b/includes/MimeMagic.php
 @@ -41,7 +41,7 @@ image/x-bmp bmp
  image/gif gif
  image/jpeg jpeg jpg jpe
  image/png png
 -image/svg+xml image/svg svg
 +image/svg+xml svg
  image/tiff tiff tif
  image/vnd.djvu image/x.djvu image/x-djvu djvu
  image/x-portable-pixmap ppm

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2010-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

--- Comment #1 from Thomas Bleher thomasble...@gmx.de 2010-12-19 16:38:27 UTC 
---
I tracked the problem down a bit, and it seems to be a problem in
includes/MimeMagic.php.

MimeMagic has a list of predefined mime types, with assorted file extensions.
The code that parses the types treats everything up to the first space as the
mime type, everything behind that as a file extension. As you can see from the
diff below, the svg line does not follow this convention, leading to an
extension of image/svg.
This blows up in UploadStash.

With the change below, I can upload svg files again. But I'm not sure what the
correct fix is here, as this particular line is in the code since r21411, from
April 2007.

--- a/includes/MimeMagic.php
+++ b/includes/MimeMagic.php
@@ -41,7 +41,7 @@ image/x-bmp bmp
 image/gif gif
 image/jpeg jpeg jpg jpe
 image/png png
-image/svg+xml image/svg svg
+image/svg+xml svg
 image/tiff tiff tif
 image/vnd.djvu image/x.djvu image/x-djvu djvu
 image/x-portable-pixmap ppm

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 26367] Uploading fails with internal error: key 'ljg40qoexthbj3vu8f9875e5uspj37n.' is not in a proper format

2010-12-19 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=26367

Raimond Spekking raimond.spekk...@gmail.com changed:

   What|Removed |Added

 CC||marc.aurel.visc...@googlema
   ||il.com

--- Comment #2 from Raimond Spekking raimond.spekk...@gmail.com 2010-12-19 
18:13:24 UTC ---
*** Bug 26368 has been marked as a duplicate of this bug. ***

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l