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

LastModifiedCachingWire.java:141-142: Evict cache entry if If-Modified-Since... #120

Closed
dmarkov opened this issue Jan 19, 2016 · 14 comments
Closed

Comments

@dmarkov
Copy link

dmarkov commented Jan 19, 2016

Puzzle 90-f5d7f05d in src/main/java/com/jcabi/http/wire/LastModifiedCachingWire.java:141-142 has to be resolved: Evict cache entry if If-Modified-Since request responded with HTTP_OK code and no Last-Modified header.... The puzzle was created by Igor Piddubnyi on 15-Jan-2016.

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.


@dmarkov
Copy link
Author

dmarkov commented Jan 20, 2016

@rajithd could you please pick this up? This article explains how we work. Any technical questions you may ask right here

The budget of this task is 30 mins. This is exactly how much time will be compensated, when the task is completed. Read about our Definition of Done

@cvrebert
Copy link

Basically, if the new response lacks a Last-Modified header, implying it's not cacheable, then there's no point in keeping the old response (which is outdated) in the cache. (And obviously the new response can't be cached since it lacks the required header.)

@dmarkov
Copy link
Author

dmarkov commented Feb 5, 2016

@rajithd the ticket is with you for 15 days already. If it is not finished in the next 48 hours, it will be re-assigned to someone else, see No Obligations principle, this article should help if you're stuck; as a reminder, don't forget to remove the @todo puzzle from the source code; -30 added to your rating, current score is: -120

@dmarkov
Copy link
Author

dmarkov commented May 16, 2016

@westonized can you please help? Keep in mind this. If you have any technical questions, don't hesitate to ask right here... The cost of this task is 30 mins (this is exactly how much will be paid, not less not more), when the task is done

westonal pushed a commit to westonal/jcabi-http that referenced this issue May 16, 2016
westonal pushed a commit to westonal/jcabi-http that referenced this issue May 16, 2016
…ded with HTTP_OK code and no Last-Modified header.
westonal pushed a commit to westonal/jcabi-http that referenced this issue May 17, 2016
westonal pushed a commit to westonal/jcabi-http that referenced this issue May 17, 2016
westonal pushed a commit to westonal/jcabi-http that referenced this issue May 17, 2016
westonal pushed a commit to westonal/jcabi-http that referenced this issue May 18, 2016
westonal pushed a commit to westonal/jcabi-http that referenced this issue May 18, 2016
westonal pushed a commit to westonal/jcabi-http that referenced this issue May 18, 2016
westonal pushed a commit to westonal/jcabi-http that referenced this issue May 18, 2016
westonal pushed a commit to westonal/jcabi-http that referenced this issue May 19, 2016
Update name
Add todo to specify behavior I feel is unspecified at this time
@westonal
Copy link
Contributor

@dmarkov The work is done and reviewed, can you follow this up for closure please? PR #134

@dmarkov
Copy link
Author

dmarkov commented May 26, 2016

@dmarkov The work is done and reviewed, can you follow this up for closure please? PR #134

@westonized PR is still open. once it's merged, we'll close this ticket

@westonal
Copy link
Contributor

westonal commented May 29, 2016

@dmarkov I know, it's waiting on yegor, 10 days now, is this normal? Has nobody else got any authority to merge? Aside from not gettting paid, I've got 2 task limit and both are waiting on yegor like this.

@dmarkov
Copy link
Author

dmarkov commented Jun 1, 2016

@dmarkov I know, it's waiting on yegor, 10 days now, is this normal? Has nobody else got any authority to merge? Aside from not gettting paid, I've got 2 task limit and both are waiting on yegor like this.

@westonized @yegor256 could you please help us here?

@westonal
Copy link
Contributor

@dmarkov PR's merged so can this get closed and I get paid?

@dmarkov
Copy link
Author

dmarkov commented Jun 20, 2016

@yegor256 2 puzzles were created here: 120-e6bf947f/#139, 120-2756c8f7

@dmarkov
Copy link
Author

dmarkov commented Jun 21, 2016

@yegor256 Done, the task is closed

@dmarkov dmarkov closed this as completed Jun 21, 2016
@dmarkov
Copy link
Author

dmarkov commented Jun 21, 2016

@westonized thank you, added 30 mins to your acc, payment num is AP-8M112932NC376593R, 839 hours and 48 mins spent in total to complete... +30 added to your rating, current score is: +45

@0pdd
Copy link

0pdd commented Jul 4, 2022

@dmarkov 2 puzzles #334, #335 are still not solved.

@0pdd
Copy link

0pdd commented Jul 19, 2022

@dmarkov 2 puzzles #unknown, #unknown are still not solved; solved: #334, #335.

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

5 participants