You are viewing a plain text version of this content. The canonical link for it is here.
Posted to slide-dev@jakarta.apache.org by ju...@apache.org on 2003/08/04 13:09:26 UTC

cvs commit: jakarta-slide/testsuite/testsuite/junit/src/org/apache/slide/testsuite/testtools/tprocessor TProcessors.java

juergen     2003/08/04 04:09:26

  Modified:    testsuite/testsuite/junit/src/org/apache/slide/testsuite/testtools/tprocessor
                        TProcessors.java
  Log:
  T-processor handles the encoding: (undo the changes in setPath: all urls are now expected to be url encoded by the application.)
  
  Revision  Changes    Path
  1.64      +12 -6     jakarta-slide/testsuite/testsuite/junit/src/org/apache/slide/testsuite/testtools/tprocessor/TProcessors.java
  
  Index: TProcessors.java
  ===================================================================
  RCS file: /home/cvs/jakarta-slide/testsuite/testsuite/junit/src/org/apache/slide/testsuite/testtools/tprocessor/TProcessors.java,v
  retrieving revision 1.63
  retrieving revision 1.64
  diff -u -r1.63 -r1.64
  --- TProcessors.java	31 Jul 2003 16:15:38 -0000	1.63
  +++ TProcessors.java	4 Aug 2003 11:09:26 -0000	1.64
  @@ -998,7 +998,13 @@
        */
       private  HttpMethod fillHeader (Element header, HttpMethod method){
           Header headerToSet = getResponseHeader(header);
  -        method.setRequestHeader(headerToSet.getName(), headerToSet.getValue());
  +        String name  = headerToSet.getName();
  +        String value = headerToSet.getValue();
  +        if (name.equalsIgnoreCase("destination")) {
  +            value = URLUtil.URLEncode(value, "utf-8");
  +        }
  +
  +        method.setRequestHeader(name, value);
           return method;
       }
   
  @@ -1725,7 +1731,7 @@
           throws Exception {
           HttpMethod result = HttpMethodFactory(methodName);
           if (result != null){
  -            result.setPath(path);
  +            result.setPath(URLUtil.URLEncode(path, "utf-8"));
           }
           return result;
       }