[Bug 61176] Content-Encoding gzip will prevent cache manager to cache embedded resources

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=61176

Felix Schumacher <fe...@internetallee.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #8 from Felix Schumacher <fe...@internetallee.de> ---
@thomas, could you test, if current trunk works for you?

This will be included in JMeter version 3.3.

Date: Thu Jul 13 17:43:15 2017
New Revision: 1801854

URL: http://svn.apache.org/viewvc?rev=1801854&view=rev
Log:
We are not not doing any real work here and it simplifies our code.

In preparation of Bugzilla Id: 61176 (and github pr #298)

Modified:
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/CacheManager.java

Date: Thu Jul 13 17:44:57 2017
New Revision: 1801855

URL: http://svn.apache.org/viewvc?rev=1801855&view=rev
Log:
Use log string templates instead of string concatenation.

This should make the code a bit more readable.

In preparation of Bugzilla Id: 61176 (and github pr #298)

Modified:
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/CacheManager.java
Date: Thu Jul 13 17:51:49 2017
New Revision: 1801856

URL: http://svn.apache.org/viewvc?rev=1801856&view=rev
Log:
Extract code into local private methods.

Extract the various methods for calculation of the expiration date into
smaller methods in an attempt to make code more readable.

In preparation of Bugzilla Id: 61176 (and github pr #298)

Modified:
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/CacheManager.java

Date: Thu Jul 13 17:53:29 2017
New Revision: 1801857

URL: http://svn.apache.org/viewvc?rev=1801857&view=rev
Log:
Rename local variable to comply with java naming conventions.

In preparation of Bugzilla Id: 61176 (and github pr #298)

Modified:
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/CacheManager.java

Date: Thu Jul 13 17:54:29 2017
New Revision: 1801858

URL: http://svn.apache.org/viewvc?rev=1801858&view=rev
Log:
Reformat to make it easier to read.

In preparation of Bugzilla Id: 61176 (and github pr #298)

Modified:
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/CacheManager.java
Date: Thu Jul 13 18:23:05 2017
New Revision: 1801859

URL: http://svn.apache.org/viewvc?rev=1801859&view=rev
Log:
Implement caching of requests with "vary" header.

Bugzilla Id: 61176 (closes #298 on github)

Modified:
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/control/CacheManager.java
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPHC4Impl.java
   
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPJavaImpl.java
   
jmeter/trunk/test/src/org/apache/jmeter/protocol/http/control/TestCacheManagerBase.java
    jmeter/trunk/xdocs/changes.xml
    jmeter/trunk/xdocs/usermanual/component_reference.xml

-- 
You are receiving this mail because:
You are the assignee for the bug.