We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Expires.Hour class does absolutely nothing. It does not modify the behavior of the object it wraps in any way.
Is this intended behavior? If so, what's the purpose of this class?
takes/src/main/java/org/takes/misc/Expires.java
Lines 100 to 103 in 925e1a2
The text was updated successfully, but these errors were encountered:
@monomonedula
It's not possible to use Expires header for relative expiration time (which seems to be an intent here) . Cache-Control should be used instead https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Cache-Control#expiration
Expires
Cache-Control
And the main use case for Expires.Date looks like that:
Expires.Date
new Expires.Date( System.currentTimeMillis() + TimeUnit.HOURS.toMillis(1L) )
So a proper solution would be to implement Cache-Control header, or at least complete Expires.Hour implementation with Expires.Date as a base class
Expires.Hour
@paulodamaso WDYT?
Sorry, something went wrong.
No branches or pull requests
Expires.Hour class does absolutely nothing. It does not modify the behavior of the object it wraps in any way.
Is this intended behavior? If so, what's the purpose of this class?
takes/src/main/java/org/takes/misc/Expires.java
Lines 100 to 103 in 925e1a2
The text was updated successfully, but these errors were encountered: