Modify

Opened 10 years ago

Closed 9 years ago

Last modified 9 years ago

#10139 closed enhancement (fixed)

improve HTTP caching

Reported by: bastiK Owned by: team
Priority: normal Milestone: 15.05
Component: Core Version:
Keywords: Cc:

Description

JOSM does not properly use HTTP mechanisms for caching, like If-Modified-Since and If-None-Match (etag).

For TMS, there is some code in OsmFileCacheTileLoader, but I think it isn't called as intended.

Attachments (0)

Change History (10)

comment:1 by bastiK, 10 years ago

In 7242/josm:

add support for If-Modified-Since header in MirroredInputStream (see #10139)

use this for /maps request (sever support has been added) and change update
intervall from 7 days to 1 day

comment:3 by bastiK, 10 years ago

In 7248/josm:

reworked MirroredInputStream (renamed to CachedFile):

  • no more awkwardly open and close InputStream if you just want the underlying file (e.g. to get file inside zip file)
  • make it easier to add configuration parameters, without having endless list of parameters for the constructor (Factory style, similar to ImageProvider)

breaks plugins; see #10139

comment:4 by bastiK, 10 years ago

Fixed plugins in [o30495], [o30496].

comment:5 by stoecker, 10 years ago

Wha, one of my major improvements renamed. Help. Help...

comment:6 by bastiK, 10 years ago

The class is gone, but your code lives on. :)

comment:7 by stoecker, 10 years ago

It was one of the few classes created by me, most stuff I did is inside existing classes. At least I found another one which is not only a helper. I wont tell you which or you'll kill it as well :-)

This reminds me how long I care for JOSM and it's fine that it is still actively developed (mainly without me).

comment:8 by Don-vip, 10 years ago

is there still something to do on this ticket?

comment:9 by bastiK, 10 years ago

Yes, HTTP caching for TMS requests needs to be investigated and probably improved.

comment:10 by wiktorn, 9 years ago

Resolution: fixed
Status: newclosed

HTTP caching for TMS was already improved in April/May.

comment:11 by Don-vip, 9 years ago

Milestone: 15.05

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.