[MAGNOLIA-3681] Cache should remove streaming files when it's done using them Created: 05/May/11 Updated: 05/May/14 Resolved: 09/Dec/11 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | cache |
| Affects Version/s: | 4.4.3 |
| Fix Version/s: | 4.4.6 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Tobias Mattsson | Assignee: | Milan Divilek |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||
| Description |
|
The cache module leave behind files on tmp directory that it use for streaming large files. This issue is reproduce in Windows OS. How to reproduce: It was discussed This forum page |
| Comments |
| Comment by Tobias Mattsson [ 07/Dec/11 ] |
|
If the file is deleted when the cache entry is replayed then its only useable once and subsequent requests that get that entry from the cache will fail. So such an entry should never be placed in the cache. I haven't tested it, should be easily tested by adding a second call in the integration test. |
| Comment by Tobias Mattsson [ 09/Dec/11 ] |
|
Closing after review, all is ok, the file is removed by the first request and subsequent requests see the entry in cache and the proceeds with rendering without using a file or response data from cache. |