[Issue 594] New - Create DeferredTransportTube by default

[jitu-4]

https://jax-ws.dev.java.net/issues/show_bug.cgi?id=594
                 Issue #|594
                 Summary|Create DeferredTransportTube by default
               Component|jax-ws
                 Version|2.1.4
                Platform|All
              OS/Version|All
                     URL|
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P3
            Subcomponent|runtime
             Assigned to|jitu
             Reported by|jitu






------- Additional comments from jitu@... Wed Jul 23 21:01:14 +0000 2008 -------
See http://forums.java.net/jive/thread.jspa?messageID=280862

This allows to set the address dynamically via the Request context.

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@...
For additional commands, e-mail: issues-help@...

[jitu-4]

https://jax-ws.dev.java.net/issues/show_bug.cgi?id=594



User jitu changed the following:

                What    |Old value                 |New value
================================================================================
                  Status|NEW                       |RESOLVED
--------------------------------------------------------------------------------
                Keywords|                          |2.1.4.1
--------------------------------------------------------------------------------
              Resolution|                          |FIXED
--------------------------------------------------------------------------------
        Target milestone|not determined            |2.1.5
--------------------------------------------------------------------------------




------- Additional comments from jitu@... Wed Jul 23 21:04:01 +0000 2008 -------
File [changed]: ClientTubeAssemblerContext.java
Url:
https://jax-ws-sources.dev.java.net/source/browse/jax-ws-sources/jaxws-ri/rt/src/com/sun/xml/ws/api/pipe/ClientTubeAssemblerContext.java?r1=1.9&r2=1.9.2.1
Delta lines:  +4 -12
--------------------
--- ClientTubeAssemblerContext.java 2008-03-25 23:33:12+0000 1.9
+++ ClientTubeAssemblerContext.java 2008-07-02 21:46:58+0000 1.9.2.1
@@ -245,19 +245,11 @@
     public Tube createTransportTube() {
         ClassLoader cl = Thread.currentThread().getContextClassLoader();
 
-        // wsgen generates a WSDL with the address attribute that says
"REPLACE_WITH_ACTUAL_URL".
-        // while it's technically correct to reject such address (since there's
no transport registered
-        // with it), it's desirable to allow the user a benefit of doubt, and
wait until the runtime
-        // to see if the user configures the endpoint address through request
context.
+        // The application may configure the endpoint address through request
context
+        // using {@link BindingProvider#ENDPOINT_ADDRESS_PROPERTY}. Let us
+        // defer the creation of actual transport until the service invocation,
         // DeferredTransportPipe is used for this purpose.
-        //
-        // Ideally, we shouldn't have @address at all for such cases, but due
to the backward
-        // compatibility and the fact that this attribute is mandatory, we have
no option but
-        // to check for REPLACE_WITH_ACTUAL_URL.
-        if(address.toString().equals("") ||
address.toString().equals("REPLACE_WITH_ACTUAL_URL"))
             return new DeferredTransportPipe(cl,this);
-
-        return TransportTubeFactory.create(cl, this);
     }
 
     /**



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@...
For additional commands, e-mail: issues-help@...