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

LastModifiedCachingWireTest.java:204-213: Confirm cache... #335

Closed
0pdd opened this issue Jul 4, 2022 · 1 comment
Closed

LastModifiedCachingWireTest.java:204-213: Confirm cache... #335

0pdd opened this issue Jul 4, 2022 · 1 comment

Comments

@0pdd
Copy link

0pdd commented Jul 4, 2022

The puzzle 120-04206dcf from #120 has to be resolved:

* @todo #120:30min Confirm cache clearing behaviour in all non-OK responses
* Non-OK behaviour was not specified in #120, so for example, if the
* response is 404 as below, does it make any sense to keep the item in
* cache? Is it likely a server will respond 404, and then later the exact
* unmodified content is available again. I think they all need to be
* thought about, another dubious response might be 301 Moved Permanently,
* or 410 Gone etc. Or, personally I think all non-OK and OK responses
* should behave the same WRT to clearing the cache as the cache value is
* so unlikely to be returned in future.
* @throws Exception If fails

The puzzle was created by @rultor on 03-Jul-22.

Estimate: 30 minutes, role: DEV.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

0pdd referenced this issue Jul 4, 2022
Bumps [jsoup](https://github.com/jhy/jsoup) from 1.15.1 to 1.15.2.
- [Release notes](https://github.com/jhy/jsoup/releases)
- [Changelog](https://github.com/jhy/jsoup/blob/master/CHANGES)
- [Commits](jhy/jsoup@jsoup-1.15.1...jsoup-1.15.2)

---
updated-dependencies:
- dependency-name: org.jsoup:jsoup
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@0crat 0crat added the 0crat/new label Jul 4, 2022
@0pdd
Copy link
Author

0pdd commented Jul 19, 2022

The puzzle 120-04206dcf has disappeared from the source code, that's why I closed this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants