You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by to...@apache.org on 2003/12/15 08:47:42 UTC
cvs commit: cocoon-2.1/src/blocks/proxy/java/org/apache/cocoon/generation WebServiceProxyGenerator.java
tony 2003/12/14 23:47:42
Modified: src/blocks/proxy/java/org/apache/cocoon/generation
WebServiceProxyGenerator.java
Log:
fixed parameter named 'null' being added to request
Revision Changes Path
1.5 +15 -2 cocoon-2.1/src/blocks/proxy/java/org/apache/cocoon/generation/WebServiceProxyGenerator.java
Index: WebServiceProxyGenerator.java
===================================================================
RCS file: /home/cvs/cocoon-2.1/src/blocks/proxy/java/org/apache/cocoon/generation/WebServiceProxyGenerator.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- WebServiceProxyGenerator.java 25 Sep 2003 19:59:24 -0000 1.4
+++ WebServiceProxyGenerator.java 15 Dec 2003 07:47:42 -0000 1.5
@@ -228,7 +228,14 @@
String submitQryString = method.getQueryString();
// set final web service query string
- method.setQueryString(urlQryString + "&" + submitQryString);
+
+ // sometimes the querystring is null here...
+ if (null == urlQryString) {
+ method.setQueryString(submitQryString);
+ } else {
+ method.setQueryString(urlQryString + "&" + submitQryString);
+ }
+
} // if there are submit parameters
byte[] response = null;
@@ -280,6 +287,12 @@
config = new HostConfiguration();
}
+
+ /* TODO: fixme!
+ * When the specified source sent to the wsproxy is not "http"
+ * (e.g. "cocoon:/"), the HttpClient throws an exception. Does the source
+ * here need to be resolved before being set in the HostConfiguration?
+ */
try {
uri = new URI(this.source);
host = uri.getHost();