Re: #153 Cookie.parse fails in parsing

Lawrence Schapker <[email protected]>
Newsgroups gmane.comp.java.grinder.user
Message-ID <[email protected]>
junoyoon <junoyoon@...> writes:

> 
> Hi. I had the same issue before.. 
> so I had to go workaround this by modifying Cookie.java in the underlying
> HTTPClient.
> 
> grinder-3.9.1-patch.jar
> <http://grinder.996249.n3.nabble.com/file/n8719/grinder-3.9.1-patch.jar>  
> 
> Please put above in the head of classpath and run agents again.
> Even though this jar file contains -3.9.1 in the file name, I believe this
> will work in 3.11 as well.
> 
> This patch includes the
> https://github.com/nhnopensource/grinder-
patch/blob/master/src/main/java/HTTPClient/Cookie.java#L265
> fix.
> 
> --
> View this message in context: http://grinder.996249.n3.nabble.com/153-
Cookie-parse-fails-in-parsing-tp8718p8719.html
> Sent from the Grinder - User mailing list archive at Nabble.com.
> 
> --------------------------------------------------------------------------
----
> Sponsored by Intel(R) XDK 
> Develop, test and display web and hybrid apps with a single code base.
> Download it for free now!
> http://pubads.g.doubleclick.net/gampad/clk?
id=111408631&iu=/4140/ostg.clktrk
> 

Thank you junoyoon, but unfortunately that patch did not work either.

Since I have not yet received junoyoon's email via the distribution, I'm 
seeing this (and replying) via gmane.org.

The issue I believe is that "HttpOnly" and "Secure" do not have an "=" AND 
there is no code to process the "HttpOnly" value prior to searching for the 
"=".  I took the source code within the #153 patch jar and added code that 
processes the "HttpOnly" value (just like the "Secure") and the issue was 
resolved.  Don't know if it is actually "correct", but I can now "move on".

I can provide the changes done to Cookie.java so that they may be added to 
the main line code, but I need to understand how you want it (I'm a 
newbie in providing opensource code changes.  I just need to understand how 
it should be done to make things easier for everyone).

Thanks to everyone!

Lawrence Schapker
Senior Performance Engineer
Salesforce.com




------------------------------------------------------------------------------
Sponsored by Intel(R) XDK 
Develop, test and display web and hybrid apps with a single code base.
Download it for free now!
http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.