[MAGNOLIA-273] Caching does not work after new thread model. request object no longer valid. Created: 01/Feb/05 Updated: 27/Apr/05 Resolved: 27/Apr/05 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | 2.01, 2.1 Final |
| Fix Version/s: | 2.1 Final |
| Type: | Bug | Priority: | Major |
| Reporter: | Erik Gulliksen | Assignee: | Sameer Charles |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
(In general)Windows XP and 2000, version 2.0 distribution with updated lib's (equal to 2.01) |
||
| Attachments: |
|
| 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
|
| Date of First Response: |
| Description |
|
Waiting for update... Here is the last mailthread: Good -Erik ----- Original Message ----- > Perfect! >>> >>> >>> >>> |
| Comments |
| Comment by Sameer Charles [ 01/Feb/05 ] |
|
Fixed on current trunk |
| Comment by Erik Gulliksen [ 01/Feb/05 ] |
|
Sweet =) Is it safe to use in 2.01 too? |
| Comment by Sameer Charles [ 01/Feb/05 ] |
|
updated tested, it works for 2.0/2.01 as well, but you need to merge it. changes: that's all |
| Comment by Erik Gulliksen [ 01/Feb/05 ] |
|
Works great =) |
| Comment by Andreas Weder [ 01/Feb/05 ] |
|
In order to retrofit this fix into the 2.01 code base, do I have to get CacheHandler from the trunk and the EntryServlet from you, Erik? Also, if I put CacheHandler into my 2.01 code base, I do get errors (SecureURI does not exist). |
| Comment by Erik Gulliksen [ 02/Feb/05 ] |
|
I use the "106 version" from 2.01 of CacheHandler (attatched too just in case). The new for 2.1 trunk is altered and u get the error with some class referances (I think they moved that class to a new package..?) Well anyway the 2 files attatched is what i used for a patch on 2.01.. Hope this clears it out =) |
| Comment by Andreas Weder [ 03/Feb/05 ] |
|
I've checked your recommended version of CacheHandler with the one currently in 2.01. It looks like the two are equal. I thus only updated EntryServlet. But I'm not sure this helped. I had caching turned on so far on my production machine. What I noticed was that the page seem to roughly take the same time to build as on the author instance. But at least it showed every time. If I enable caching now, I get an empty page as soon as I hit the cache on the public instance. So something is wrong. What could it be? I've activated caching by setting "active" to "true", and set "domain" to "localhost:8082". My public instance is indeed running on port 8082. |
| Comment by Sameer Charles [ 27/Apr/05 ] |
|
on svn |