Modify

Opened 14 years ago

Closed 9 years ago

Last modified 9 years ago

#6103 closed defect (fixed)

Do not close changeset after "cancelled upload"

Reported by: chriscf Owned by: team
Priority: normal Milestone: 16.02
Component: Core Version: tested
Keywords: Cc:

Description (last modified by Don-vip)

A transient fault in the connection can result in a CancelledUploadException, which leaves the upload incomplete, and if marked for closure the changeset is closed. In such a situation, it probably shouldn't be closed until the upload completes successfully.

POST http://www.openstreetmap.org/api/0.6/changeset/xxxxxxx/upload... Proxy Error
Waiting 10 seconds ... org.openstreetmap.josm.io.OsmTransferCancelledException
        at org.openstreetmap.josm.io.OsmApi.sleepAndListen(OsmApi.java:484)
        at org.openstreetmap.josm.io.OsmApi.sendRequest(OsmApi.java:569)
        at org.openstreetmap.josm.io.OsmApi.sendRequest(OsmApi.java:508)
        at org.openstreetmap.josm.io.OsmApi.uploadDiff(OsmApi.java:458)
        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(Unknown Source)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
        at java.lang.Thread.run(Unknown Source)
PUT http://www.openstreetmap.org/api/0.6/changeset/7529595/close... OK

Attachments (0)

Change History (3)

comment:1 by Don-vip, 12 years ago

Description: modified (diff)
Reporter: changed from anonymous to chriscf

comment:2 by simon04, 9 years ago

Resolution: fixed
Status: newclosed

No longer reproducible with r9514

What I tested:

  1. Get hold of an open changeset
  2. Add 100's of ways (to have some time to cancel)
  3. Upload
  4. Cancel soon
  5. The console shows
    INFO: POST http://api06.dev.openstreetmap.org/api/0.6/changeset/70175/upload (10.3 kB) ...
    INFO: POST http://api06.dev.openstreetmap.org/api/0.6/changeset/70175/upload -> !!!
    INFO: Ignoring caught exception because upload is canceled. Exception is: org.openstreetmap.josm.io.OsmTransferException: java.net.SocketException: Socket closed
    
  6. The changeset is still open

comment:3 by simon04, 9 years ago

Milestone: 16.02

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.