Skip to content
New issue

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

Last-Modified comparison needs to account for 1 second precision #1788

Closed
hueniverse opened this issue Jul 16, 2014 · 3 comments
Closed

Last-Modified comparison needs to account for 1 second precision #1788

hueniverse opened this issue Jul 16, 2014 · 3 comments
Assignees
Milestone

Comments

@hueniverse
Copy link
Contributor

@hueniverse hueniverse commented Jul 16, 2014

No description provided.

@hueniverse hueniverse added this to the 6.1.0 milestone Jul 16, 2014
@hueniverse hueniverse added the bug label Jul 16, 2014
@hueniverse hueniverse self-assigned this Jul 16, 2014
@hueniverse hueniverse changed the title Last-Modified test need to account for 1 second precision Last-Modified comparison needs to account for 1 second precision Jul 16, 2014
@hueniverse
Copy link
Contributor Author

@hueniverse hueniverse commented Jul 16, 2014

Hmm. Not sure about this anymore...

@hueniverse hueniverse added non-issue and removed bug labels Jul 16, 2014
@hueniverse hueniverse reopened this Jul 16, 2014
hueniverse pushed a commit that referenced this issue Jul 16, 2014
@hueniverse hueniverse closed this Jul 16, 2014
@bleupen
Copy link

@bleupen bleupen commented Oct 30, 2014

If my request has an if-modified-since header of "Thu Oct 30 2014 13:43:51 GMT-0400" and i generate a response with a last-modified header of "2014-10-30T13:43:51.899Z" (ie after but within the same second), hapi will send a 304. Is this the correct behavior?

@hueniverse
Copy link
Contributor Author

@hueniverse hueniverse commented Oct 30, 2014

Header comparison is at 1 second granularity.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants