Bug 27397 - MultiUploader berücksichtigt fehlerhaften Dateiupload nicht
MultiUploader berücksichtigt fehlerhaften Dateiupload nicht
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P3 normal (vote)
: UCS 3.1
Assigned To: Florian Best
Lukas Walter
: interim-2
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-31 14:32 CEST by Alexander Kläser
Modified: 2012-12-12 21:09 CET (History)
1 user (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Usability
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2012-05-31 14:32:53 CEST
Mir ist gerade aufgefallen, dass der MultiUploader Fehler beim Dateiupload nicht richtig handhabt. Ich erhielt einen 503er-Statuscode (=forbidden), die Datei wurde aber dennoch in die Liste der hochgeladenen Dateien eingetragen.
Comment 1 Alexander Kläser univentionstaff 2012-05-31 14:37:59 CEST
(In reply to comment #0)
> ... Ich erhielt einen 503er-Statuscode (=forbidden) ...

Ich meinte Fehlercode 403 :)
Comment 2 Florian Best univentionstaff 2012-06-21 09:45:07 CEST
(In reply to comment #0)
> Mir ist gerade aufgefallen, dass der MultiUploader Fehler beim Dateiupload
> nicht richtig handhabt. Ich erhielt einen 503er-Statuscode (=forbidden), die
> Datei wurde aber dennoch in die Liste der hochgeladenen Dateien eingetragen.
Das ist wahrscheinlich mit Bug #27415 behoben ?!
Comment 3 Stefan Gohmann univentionstaff 2012-07-17 17:09:37 CEST
UCS 3.1 will be the next release.
Comment 4 Alexander Kläser univentionstaff 2012-09-03 16:37:54 CEST

*** This bug has been marked as a duplicate of bug 27415 ***
Comment 5 Florian Best univentionstaff 2012-10-16 09:22:33 CEST
Momentan funktioniert im IE8 ein Dateiupload.
Das Bild eines Benutzers wird nicht angezeigt.
Die Fehlererkennung funktioniert nicht:
Es wird ein Download-Dialog angezeigt beim Fehlschlag (503 und 403).
Comment 6 Florian Best univentionstaff 2012-10-16 14:21:56 CEST
In chromium funktioniert die Fehler-Benachrichtigung auch noch nicht, da versucht wird den server-response als json auszuwerten.
Comment 7 Alexander Kläser univentionstaff 2012-10-16 15:15:59 CEST
Es scheint das wir folgende Änderung für einen Fix noch brauchen:

Patch für dojox/form/uploader/plugins/HTML5:
====================
try {
    this.onComplete(JSON.parse(xhr.responseText.replace(/^\{\}&&/,'')));
} catch(e) {
    this.onComplete(xhr.responseText);
}
====================
→ verhindert Errors beim Parsen der Rückgabewerte (können HTML-Dokumente sein bei 503-Fehlern).
Comment 8 Florian Best univentionstaff 2012-10-16 15:29:35 CEST
(In reply to comment #7)
> Es scheint das wir folgende Änderung für einen Fix noch brauchen:
> 
> Patch für dojox/form/uploader/plugins/HTML5:
> ====================
> try {
>     this.onComplete(JSON.parse(xhr.responseText.replace(/^\{\}&&/,'')));
> } catch(e) {
>     this.onComplete(xhr.responseText);
> }
> ====================
> → verhindert Errors beim Parsen der Rückgabewerte (können HTML-Dokumente sein
> bei 503-Fehlern).
Gefixt in univention-dojo (6.0.7-1)
Comment 9 Alexander Kläser univentionstaff 2012-10-17 12:36:32 CEST
(In reply to comment #5)
> Momentan funktioniert im IE8 ein Dateiupload.
> Das Bild eines Benutzers wird nicht angezeigt.
> Die Fehlererkennung funktioniert nicht:
> Es wird ein Download-Dialog angezeigt beim Fehlschlag (503 und 403).

Das funktioniert jetzt. Problem war, dass der Upload-Button deaktiviert wurde bevor die Daten verschickt wurden. Dadurch waren die Daten leer. Nun wird der Button deaktiviert nachdem die Daten versendet werden. Damit sollte alles behoben sein.


 univention-management-console-frontend (2.0.117-1) unstable; urgency=low
 .
   * Uploader: disable button after performing the update; Bug #27397
   * updated tests/uploader.html
Comment 10 Lukas Walter univentionstaff 2012-10-19 17:38:32 CEST
Getestet werden kann in UCS lediglich der normale Dateiuploader, der MultiUploader wird nur in UCS@school verwendet (das wird daher im Zuge von UCS@school 3.1 mit Bug #28818 getestet).

Der Dateiuploader funktioniert - allerdings noch nicht mit dem IE8 unter Windows 7, dazu gibt es Bug #27414.


Damit ist dieser hier VERIFIED.
Comment 11 Stefan Gohmann univentionstaff 2012-12-12 21:09:46 CET
UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

If this error occurs again, please use "Clone This Bug".