You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@servicemix.apache.org by "Guillaume Nodet (JIRA)" <ji...@apache.org> on 2008/11/27 17:28:05 UTC

[jira] Created: (SM-1708) Support WS-Addressing for the new http endpoints

Support WS-Addressing for the new http endpoints
------------------------------------------------

                 Key: SM-1708
                 URL: https://issues.apache.org/activemq/browse/SM-1708
             Project: ServiceMix
          Issue Type: New Feature
          Components: servicemix-http
            Reporter: Guillaume Nodet
         Attachments: SM-1708.patch



-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47935#action_47935 ] 

Guillaume Nodet commented on SM-1708:
-------------------------------------

Adding         servicemix-common/src/main/java/org/apache/servicemix/common/tools/wsdl/PortTypeDecorator.java
Sending        servicemix-soap2/pom.xml
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/api/Message.java
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/api/model/Binding.java
Adding         servicemix-soap2/src/main/java/org/apache/servicemix/soap/bindings/jbi
Adding         servicemix-soap2/src/main/java/org/apache/servicemix/soap/bindings/jbi/JbiConstants.java
Adding         servicemix-soap2/src/main/java/org/apache/servicemix/soap/bindings/jbi/impl
Adding         servicemix-soap2/src/main/java/org/apache/servicemix/soap/bindings/jbi/impl/Wsdl1JbiBindingImpl.java
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/core/model/AbstractBinding.java
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/interceptors/jbi/JbiOutInterceptor.java
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/interceptors/wsdl/WsdlOperationInInterceptor.java
Adding         servicemix-soap2/src/main/java/org/apache/servicemix/soap/ws/addressing/AbstractWsAddressingInterceptor.java
Adding         servicemix-soap2/src/main/java/org/apache/servicemix/soap/ws/addressing/WsAddressingInDestinationInterceptor.java
Deleting       servicemix-soap2/src/main/java/org/apache/servicemix/soap/ws/addressing/WsAddressingInInterceptor.java
Adding         servicemix-soap2/src/main/java/org/apache/servicemix/soap/ws/addressing/WsAddressingInOperationInterceptor.java
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/ws/addressing/WsAddressingOutInterceptor.java
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/ws/addressing/WsAddressingPolicy.java
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/wsdl/Wsdl1Soap11BindingFactory.java
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/wsdl/Wsdl1Soap12BindingFactory.java
Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/wsdl/Wsdl2BindingFactory.java
Adding         servicemix-soap2/src/test/java/org/apache/servicemix/soap/ws
Adding         servicemix-soap2/src/test/java/org/apache/servicemix/soap/ws/addressing
Adding         servicemix-soap2/src/test/java/org/apache/servicemix/soap/ws/addressing/WsAddressingTest.java
Adding         servicemix-soap2/src/test/java/org/apache/servicemix/tck/mock/MockComponentContext.java
Adding         servicemix-soap2/src/test/resources/org/apache/servicemix/soap/ws
Adding         servicemix-soap2/src/test/resources/org/apache/servicemix/soap/ws/addressing
Adding         servicemix-soap2/src/test/resources/org/apache/servicemix/soap/ws/addressing/HelloWorld-DOC.wsdl
Adding         servicemix-soap2/src/test/resources/org/apache/servicemix/soap/ws/addressing/addressing-request.xml
Adding         servicemix-soap2/src/test/resources/org/apache/servicemix/soap/ws/addressing/simple.wsdl
Transmitting file data ......................
Committed revision 724359.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Guillaume Nodet updated SM-1708:
--------------------------------

    Attachment: SM-1708.patch

Patch for current work on WS-Addressing

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Jean-Baptiste Onofré (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jean-Baptiste Onofré updated SM-1708:
-------------------------------------

    Attachment: SM-1708-p1.patch

First patch with only WSDL proxy (not the WS-Addressing).

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Jean-Baptiste Onofré (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47659#action_47659 ] 

Jean-Baptiste Onofré commented on SM-1708:
------------------------------------------

As seen with Guillaume, the interesting code is inside the WsAddressingInOperationInterceptor code.
It retrieves the WSDL description from the target endpoint, decorate it to make it a real SOAP WSDL, then creating a Binding from it.
It's the behavior of the old HTTP endpoints.

For the new HTTP endpoints, the SOAP Marshaler could do the same.

I'm going to patch the SOAP Marshaler and test the behavior.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Work started: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Work on SM-1708 started by Guillaume Nodet.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>            Assignee: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47937#action_47937 ] 

Guillaume Nodet commented on SM-1708:
-------------------------------------

Sending        servicemix-common/src/main/java/org/apache/servicemix/common/tools/wsdl/PortTypeDecorator.java
Transmitting file data .
Committed revision 724366.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Jean-Baptiste Onofré (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47851#action_47851 ] 

Jean-Baptiste Onofré commented on SM-1708:
------------------------------------------

OK, I'm going to open a new JIRA and clean the patch.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Jean-Baptiste Onofré (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47814#action_47814 ] 

Jean-Baptiste Onofré commented on SM-1708:
------------------------------------------

The "old" HTTP endpoint extends SoapEndpoint abstract class.
In this class, the method responsible of the WSDL proxy is retrieveProxiedEndpointDefinition().
It creates a WSDL definition for a consumer endpoint by loading the target endpoint definition and adding HTTP binding informations to it.

I will provide a first patch for the soap-consumer and consumer "new" endpoints with the same behavior.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47938#action_47938 ] 

Guillaume Nodet commented on SM-1708:
-------------------------------------

Sending        src/main/java/org/apache/servicemix/http/HttpEndpoint.java
Deleting       src/main/java/org/apache/servicemix/http/tools
Adding         src/test/java/org/apache/servicemix/http/addressing
Adding         src/test/java/org/apache/servicemix/http/addressing/AddressingConsumerTest.java
Adding         src/test/resources/org/apache/servicemix/http/Echo.wsdl
Adding         src/test/resources/org/apache/servicemix/http/HelloWorld-DOC-soap12.wsdl
Transmitting file data ...
Committed revision 724368.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>            Assignee: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Jean-Baptiste Onofré (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47878#action_47878 ] 

Jean-Baptiste Onofré commented on SM-1708:
------------------------------------------

I have created SM-1721 for the WSDL proxy.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Guillaume Nodet reassigned SM-1708:
-----------------------------------

    Assignee: Guillaume Nodet

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>            Assignee: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Issue Comment Edited: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Jean-Baptiste Onofré (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47659#action_47659 ] 

nanthrax edited comment on SM-1708 at 12/1/08 3:33 AM:
-------------------------------------------------------------------

As seen with Guillaume, the interesting code is inside the WsAddressingInOperationInterceptor class.
It retrieves the WSDL description from the target endpoint, decorate it to make it a real SOAP WSDL, then creating a Binding from it.
It's the behavior of the old HTTP endpoints.

For the new HTTP endpoints, the SOAP Marshaler could do the same.

I'm going to patch the SOAP Marshaler and test the behavior.

      was (Author: nanthrax):
    As seen with Guillaume, the interesting code is inside the WsAddressingInOperationInterceptor code.
It retrieves the WSDL description from the target endpoint, decorate it to make it a real SOAP WSDL, then creating a Binding from it.
It's the behavior of the old HTTP endpoints.

For the new HTTP endpoints, the SOAP Marshaler could do the same.

I'm going to patch the SOAP Marshaler and test the behavior.
  
> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Guillaume Nodet resolved SM-1708.
---------------------------------

       Resolution: Fixed
    Fix Version/s: servicemix-http-2008.02
                   servicemix-shared-2008.02

Sending        src/main/java/org/apache/servicemix/http/endpoints/HttpConsumerEndpoint.java
Transmitting file data .
Committed revision 726136.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>            Assignee: Guillaume Nodet
>             Fix For: servicemix-shared-2008.02, servicemix-http-2008.02
>
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47936#action_47936 ] 

Guillaume Nodet commented on SM-1708:
-------------------------------------

Sending        servicemix-soap2/src/main/java/org/apache/servicemix/soap/core/model/AbstractBinding.java
Transmitting file data .
Committed revision 724362.

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (SM-1708) Support WS-Addressing for the new http endpoints

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/activemq/browse/SM-1708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=47846#action_47846 ] 

Guillaume Nodet commented on SM-1708:
-------------------------------------

It may be better to raise another JIRA issue for the WSDL bits.
Anyway, the patch is quite difficult to analyze because the whole file has been reformated, so it becomes very difficult to see where the real changes are.
Would it be possible to provide a cleaner patch and attach it to a newly created issue?

> Support WS-Addressing for the new http endpoints
> ------------------------------------------------
>
>                 Key: SM-1708
>                 URL: https://issues.apache.org/activemq/browse/SM-1708
>             Project: ServiceMix
>          Issue Type: New Feature
>          Components: servicemix-http
>            Reporter: Guillaume Nodet
>         Attachments: SM-1708-p1.patch, SM-1708.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.