search

indieweb / chat.indieweb.org

https://chat.indieweb.org
Apache License 2.0
15 stars 7 forks source link

Add last-modified HTTP header and support if-modified-since requests #15

Closed cweiske closed 8 years ago

cweiske commented 8 years ago

To support my search engine at indexing, I'd like chat.indieweb.org to support If-Modified-Since HTTP request headers, and it should send out Last-Modified HTTP headers for all chat log pages.

cweiske commented 8 years ago

Could you do that for day.php and today.php, too?

aaronpk commented 8 years ago

today.php is always a redirect, do you need a header on that?

day.php will be tricky but I can do that. Basically after the day is over it never changes, but for the current day it should never be cached.

cweiske commented 8 years ago

Ah, then today.php does not need any change.

day.php - your code description seems sufficient.

kevinmarks commented 8 years ago

Last-modified is still useful, especially on the lower traffic channels. You could set the cache-control policy accordingly, private with a low max-age for day, public with a very high max-age for archives

On 27 Aug 2016 9:09 am, "Aaron Parecki" notifications@github.com wrote:

today.php is always a redirect, do you need a header on that?

day.php will be tricky but I can do that. Basically after the day is over it never changes, but for the current day it should never be cached.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

aaronpk commented 8 years ago

@kevinmarks what's the use case for adding cache headers for today's logs?

cweiske commented 8 years ago

I also don't see a reason to add cache control headers now. last-modified is enough.