Opened 15 years ago

Last modified 3 years ago

#6151 new defect

If upload ends with internal server error, no warning is shown — at Initial Version

Reported by: bilbo Owned by: team
Priority: critical Milestone:
Component: Core Version: tested
Keywords: Cc:

Description

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)

Change History (0)

Note: See TracTickets for help on using tickets.