You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by ke...@apache.org on 2007/06/14 14:45:12 UTC
svn commit: r547238 - in
/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2:
description/ util/
Author: keithc
Date: Thu Jun 14 05:45:10 2007
New Revision: 547238
URL: http://svn.apache.org/viewvc?view=rev&rev=547238
Log:
Fixing HTTPLocation resolving. Removing the incorrect / appended the the endpoint address (This was a bogus fix to the original prolem).
The resolution of the HTTPLocation against the base uri should produce a uri such as http://XXXXX/axis2/services/serviceName/operationName
This is the default of axis2. The most economisc way to produce is to change the httpLocation attribute to be serviceName/operationName.
Modified:
webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBinding.java
webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBindingOperation.java
webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL11.java
webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL20.java
webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/util/WSDLSerializationUtil.java
Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBinding.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBinding.java?view=diff&rev=547238&r1=547237&r2=547238
==============================================================================
--- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBinding.java (original)
+++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBinding.java Thu Jun 14 05:45:10 2007
@@ -117,7 +117,7 @@
* @return The generated binding element
*/
public OMElement toWSDL20(OMNamespace wsdl, OMNamespace tns, OMNamespace wsoap, OMNamespace whttp,
- String interfaceName, Map nameSpaceMap, String addressingFlag) {
+ String interfaceName, Map nameSpaceMap, String addressingFlag, String serviceName) {
String property;
OMFactory omFactory = OMAbstractFactory.getOMFactory();
OMElement bindingElement;
@@ -197,7 +197,7 @@
Iterator iterator = this.getChildren();
while (iterator.hasNext()) {
AxisBindingOperation axisBindingOperation = (AxisBindingOperation) iterator.next();
- bindingElement.addChild(axisBindingOperation.toWSDL20(wsdl, tns, wsoap, whttp, type, nameSpaceMap));
+ bindingElement.addChild(axisBindingOperation.toWSDL20(wsdl, tns, wsoap, whttp, type, nameSpaceMap, serviceName));
}
WSDLSerializationUtil.addWSDLDocumentationElement(this, bindingElement, omFactory, wsdl);
return bindingElement;
Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBindingOperation.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBindingOperation.java?view=diff&rev=547238&r1=547237&r2=547238
==============================================================================
--- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBindingOperation.java (original)
+++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisBindingOperation.java Thu Jun 14 05:45:10 2007
@@ -117,7 +117,7 @@
* @return The generated binding element
*/
public OMElement toWSDL20(OMNamespace wsdl, OMNamespace tns, OMNamespace wsoap, OMNamespace whttp,
- String type, Map nameSpaceMap) {
+ String type, Map nameSpaceMap, String serviceName) {
String property;
OMFactory omFactory = OMAbstractFactory.getOMFactory();
OMElement bindingOpElement =
@@ -178,7 +178,7 @@
property = (String) this.options.get(WSDL2Constants.ATTR_WHTTP_LOCATION);
if (property != null) {
bindingOpElement.addAttribute(omFactory.createOMAttribute(
- WSDL2Constants.ATTRIBUTE_LOCATION, whttp, property));
+ WSDL2Constants.ATTRIBUTE_LOCATION, whttp, serviceName + "/" + property));
}
property = (String) this.options.get(WSDL2Constants.ATTR_WHTTP_CONTENT_ENCODING);
if (property != null) {
Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL11.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL11.java?view=diff&rev=547238&r1=547237&r2=547238
==============================================================================
--- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL11.java (original)
+++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL11.java Thu Jun 14 05:45:10 2007
@@ -783,7 +783,7 @@
OMElement httpOperation = fac.createOMElement("operation", http);
operation.addChild(httpOperation);
- httpOperation.addAttribute("location", axisOperation.getName()
+ httpOperation.addAttribute("location", axisService.getName() + "/" + axisOperation.getName()
.getLocalPart(), null);
String MEP = axisOperation.getMessageExchangePattern();
Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL20.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL20.java?view=diff&rev=547238&r1=547237&r2=547238
==============================================================================
--- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL20.java (original)
+++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService2WSDL20.java Thu Jun 14 05:45:10 2007
@@ -182,9 +182,6 @@
bindings.add(axisEndpoint.getBinding());
for (int i = 0; i < eprs.length; i++) {
String epr = eprs[i];
- if (!epr.endsWith("/")) {
- epr = epr + "/";
- }
OMElement endpointElement = axisEndpoint.toWSDL20(wsdl, tns, whttp, epr);
boolean endpointAlreadyAdded = false;
Iterator endpointsAdded = serviceElement.getChildren();
@@ -214,7 +211,8 @@
.addChild(binding.toWSDL20(wsdl, tns, wsoap, whttp,
interfaceName,
axisService.getNameSpacesMap(),
- axisService.getWSAddressingFlag()));
+ axisService.getWSAddressingFlag(),
+ axisService.getName()));
}
descriptionElement.addChild(serviceElement);
Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/util/WSDLSerializationUtil.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/util/WSDLSerializationUtil.java?view=diff&rev=547238&r1=547237&r2=547238
==============================================================================
--- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/util/WSDLSerializationUtil.java (original)
+++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/util/WSDLSerializationUtil.java Thu Jun 14 05:45:10 2007
@@ -227,8 +227,9 @@
public static OMElement generateHTTPBinding(OMFactory fac, AxisService axisService,
OMNamespace wsdl, OMNamespace whttp, OMNamespace tns) {
OMElement binding = fac.createOMElement(WSDL2Constants.BINDING_LOCAL_NAME, wsdl);
+ String serviceName = axisService.getName();
binding.addAttribute(
- fac.createOMAttribute(WSDL2Constants.ATTRIBUTE_NAME, null, axisService.getName() +
+ fac.createOMAttribute(WSDL2Constants.ATTRIBUTE_NAME, null, serviceName +
Java2WSDLConstants.HTTP_BINDING));
binding.addAttribute(fac.createOMAttribute(WSDL2Constants.INTERFACE_LOCAL_NAME, null, tns
.getPrefix() + ":" + WSDL2Constants.DEFAULT_INTERFACE_NAME));
@@ -244,7 +245,7 @@
opElement.addAttribute(fac.createOMAttribute(WSDL2Constants.ATTRIBUTE_REF, null,
tns.getPrefix() + ":" + name));
opElement.addAttribute(fac.createOMAttribute(WSDL2Constants.ATTRIBUTE_LOCATION, whttp,
- name));
+ serviceName + "/" + name));
}
return binding;
}
@@ -287,9 +288,6 @@
for (int i = 0; i < eprs.length; i++) {
String name = "";
String epr = eprs[i];
- if (!epr.endsWith("/")) {
- epr = epr + "/";
- }
if (epr.startsWith("https://")) {
name = WSDL2Constants.DEFAULT_HTTPS_PREFIX;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org