mkostrze    2004/03/23 15:23:33

  Modified:    src/protocol/http/org/apache/jmeter/protocol/http/control
                        CookieManager.java
  Log:
  Fixes a problem with cookies coming back from Oracle9iAS where
  cookies would have leading spaces in the domain portion.  For instance,
  'domain= .mydomain.com;'.  Both IE6 on W2K and Moz 1.6 on W2K and Linux
  trim the domain.
  Obtained from: jmeter-user
  Submitted by: Todd Parnell
  Reviewed by: Michal Kostrzewa
  CVS: ----------------------------------------------------------------------
  CVS: PR:
  CVS:   If this change addresses a PR in the problem report tracking
  CVS:   database, then enter the PR number(s) here.
  CVS: Obtained from:
  CVS:   If this change has been taken from another system, such as NCSA,
  CVS:   then name the system in this line, otherwise delete it.
  CVS: Submitted by:
  CVS:   If this code has been contributed to Apache by someone else; i.e.,
  CVS:   they sent us a patch or a new module, then include their name/email
  CVS:   address here. If this is your work then delete this line.
  CVS: Reviewed by:
  CVS:   If we are doing pre-commit code reviews and someone else has
  CVS:   reviewed your changes, include their name(s) here.
  CVS:   If you have not had it reviewed then delete this line.
  
  Revision  Changes    Path
  1.26      +5 -3      
jakarta-jmeter/src/protocol/http/org/apache/jmeter/protocol/http/control/CookieManager.java
  
  Index: CookieManager.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-jmeter/src/protocol/http/org/apache/jmeter/protocol/http/control/CookieManager.java,v
  retrieving revision 1.25
  retrieving revision 1.26
  diff -u -r1.25 -r1.26
  --- CookieManager.java        16 Mar 2004 19:48:57 -0000      1.25
  +++ CookieManager.java        23 Mar 2004 23:23:32 -0000      1.26
  @@ -389,7 +389,9 @@
               }
               else if (key.equalsIgnoreCase("domain"))
               {
  -                domain= nvp.substring(index + 1);
  +                //trim() is a workaround for bug in Oracle8iAS wherere
  +             //cookies would have leading spaces in the domain portion
  +             domain= nvp.substring(index + 1).trim();
                   
                   // The standard dictates domains must have a leading dot,
                   // but the new standard (Cookie2) tells us to add it if it's not
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to