search

codeformuenster / parkleitsystem-api

Ein API-Wrapper für das Parkleitsystem des Tiefbauamts Münster.
http://www.stadt-muenster.de/tiefbauamt/parkleitsystem/
Apache License 2.0
3 stars 4 forks source link

check for nil before checking cache fetch time #8

Closed ubergesundheit closed 9 years ago

ubergesundheit commented 9 years ago

@milafrerichs @djmaze Meinungen?

milafrerichs commented 9 years ago

würde einfach Time.now.to_i - @cache[:fetch_time].to_i < CACHE_MAX_AGE machen denn nil.to_i => 0

djmaze commented 9 years ago

Hmm, ich bin für @ubergesundheit's Lösung, denn jetzt musste ich einfach zu lange drüber nachdenken, ob ich @milafrerichs Lösung wirklich gut finde oder mir der Ansatz zu implizit ist ;)

milafrerichs commented 9 years ago

:) aber das != nilist dann überflüssig

Besser wäre noch eine richtige Cache Lösung mit 304 - not modified

djmaze commented 9 years ago

aber das != nilist dann überflüssig

Jau.

Besser wäre noch eine richtige Cache Lösung mit 304 - not modified

Das wäre unabhängig vom Server-seitigen Caching eine zusätzliche Funktionalität für das Browser-Caching, bei dem man die Client-Header checkt und evtl. in den Responses ein ETag zurückgibt (siehe Conditional GET support bei Rails).

Das wäre auch durchaus sinnvoll! Würde ich aber als separate Story sehen.

milafrerichs commented 9 years ago

Das wäre unabhängig vom Server-seitigen Caching eine zusätzliche Funktionalität für das Browser-Caching

jo hast natürlich recht :)