id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
6151	If upload ends with internal server error, no warning is shown	bilbo	team	"I was using upload of larger amount of elements, with uploading by parts.

I found out, that if upload ends with internal server error, then no error message is shown and there is no indication that upload went wrong (except that if you opress ""upload"" again, you get upload dialog (with objects remaining to be uploaded) instead of ""Nothing to upload"")

In such cases (Internal server error received from server), the upload should either autmatically resume, or at least warn user that something went wrong (like a messagebox), instead of silently terminating the upload (so user thinks everything was uploaded OK).

Only this message is shown on JOSM console:

{{{
POST http://www.openstreetmap.org/api/0.6/changeset/7675004/upload... Internal Server Error
Waiting 10 seconds ... org.openstreetmap.josm.io.OsmTransferCancelledException
        at org.openstreetmap.josm.io.OsmApi.sleepAndListen(OsmApi.java:455)
        at org.openstreetmap.josm.io.OsmApi.sendRequest(OsmApi.java:540)
        at org.openstreetmap.josm.io.OsmApi.sendRequest(OsmApi.java:479)
        at org.openstreetmap.josm.io.OsmApi.uploadDiff(OsmApi.java:429)
        at org.openstreetmap.josm.io.OsmServerWriter.uploadChangesInChunks(OsmServerWriter.java:163)
        at org.openstreetmap.josm.io.OsmServerWriter.uploadOsm(OsmServerWriter.java:205)
        at org.openstreetmap.josm.gui.io.UploadPrimitivesTask.realRun(UploadPrimitivesTask.java:236)
        at org.openstreetmap.josm.gui.PleaseWaitRunnable.doRealRun(PleaseWaitRunnable.java:83)
        at org.openstreetmap.josm.gui.PleaseWaitRunnable.run(PleaseWaitRunnable.java:129)
        at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
        at java.lang.Thread.run(Thread.java:662)
}}}"	defect	new	critical		Core	tested