Tickets :: [#7315] remote_import doesn't report errors when uploads fail for large images

Summary	remote_import doesn't report errors when uploads fail for large images
Queue	Ansel
Type	Bug
State	Resolved
Priority	1. Low
Owners	Chuck Hagenbuch <chuck (at) horde (dot) org>
Requester	Chuck Hagenbuch <chuck (at) horde (dot) org>
Created	09/09/2008 (83 days ago)
Due
Updated	10/03/2008 (59 days ago)
Assigned	09/22/2008 (70 days ago)
Resolved	10/03/2008 (59 days ago)
Attachments
Milestone
Patch	No

10/03/2008	Michael Rubinsky	Comment #7 State ⇒ Resolved	Reply to this comment
Chuck says this is good now :)

09/22/2008	Chuck Hagenbuch	Comment #6 State ⇒ Assigned Taken from Michael Rubinsky	Reply to this comment
I haven't had a chance to test again yet.

09/22/2008	Jan Schneider	Comment #5	Reply to this comment
Ping?

09/10/2008	Michael Rubinsky	State ⇒ Feedback

09/10/2008	Michael Rubinsky	Comment #4	Reply to this comment
We at least catch some more error cases now, although for some reason, when I hit a post_max_size error, the image still appears in the gallery. What limit were you hitting with the image size?

09/10/2008	CVS Commit	Comment #3	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/ansel/scripts/remote_import.php?r1=1.5&r2=1.6&ty=u

09/09/2008	Chuck Hagenbuch	Comment #2 Summary ⇒ remote_import doesn't report errors when uploads fail for large images	Reply to this comment
Okay, the _ was coincidental. The files were large (9+ MB), and that seems to be what did it. We should still propagate an error back to the remote script, though, so the user knows the upload failed.

09/09/2008	Chuck Hagenbuch	Comment #1 Patch ⇒ Milestone ⇒ Assigned to Michael Rubinsky Assigned to Chuck Hagenbuch Queue ⇒ Ansel Summary ⇒ remote_import seems to have trouble with files beginning with an underscore Type ⇒ Bug Priority ⇒ 1. Low State ⇒ Assigned	Reply to this comment
It reported to me that it successfully uploaded 39 images, but there are only 25 new images in the gallery I was uploading to, leaving off the 14 that started with _. Not sure where/why this happened, but I don't want to forget to investigate.