[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [ebxml-msg] Groups - ebXML Messaging TC weekly call added
Team, I probably don't have time to edit the current draft and process my earlier comments. I did manage to review section 2.6.6 and 2.7. One general remark after reading the draft is that to me it seems more descriptive rather than directive. Comments on sections 2.6.6 and 2.7:
L913-917: This paragraph is about routing Error signal messages where the section itself is on the reporting of routing failures. Routing Error messages is already covered in section 2.6.5 (as the routing of an ebMS signal message). The requirements on endpoints is covered in 2.7.3. So I propose to remove this paragraph. L934: Add sentence “If both headers are available the error MUST be sent to the URL given by the wsa:FaultTo header.” to make error reporting by intermediaries and endpoints consistent (see also section 2.7.3) L938: Add sentence “Again when both headers are available the EPR from the wsa:FaultTo header MUST be used.” L943-945: This option requires an intermediary to execute the routing function for a message as soon as it is received on the transport level, even before it is acknowledged [on the transport level]. Otherwise the error cannot be reported back on http response. I would assume that an Intermediary must always support the default option, so Intermediaries are not allowed to just store and acknowledge the message on the transport level and than process it further. Do we want such a requirement? I would propose to rename this option to something like “Synchronous reporting” §2.7.1: This section is more general than the endpoint MSH as it defines the WS-A EPR which is also relevant to the intermediary MSH. I propose to promote this section to a higher level and make this a new section 2.8 and renumber current sections 2.8 and up. Reading this section I would also think that L880-888 should be moved to this section and in their place a reference should be made to this section. L965: Insert sentence "WS-A EPR's primarily use URL's to identity endpoints." L967-969: Text here is unclear and suggests that WS-A Action is used by the routing function. Section 2.6.5 however does not specify this. Propose to rephrase text to: "Consequently a WS-A reference parameter is used to allow the routing of messages through the I-Cloud as defined in section 2.6.5." L1042: Change text to: “The Initiating message is defined as the first message sent by the Initiating MSH as defined in section 2.2.3 of the Core specification. As shown in section 2.5.2.2, in a multi-hop configuration both an endpoint MSH (cases 3a) and an Intermediairy MSH (case 3b and 4) can be the initiator of a message exchange. Three kinds of initiating messages can be distinguished:”. L1045: Change “these headers” to “this header” L1046: Add option to add an EPR and add reference to section 4.1 on how to configure EPR in Pmode? L1047: Change first sentence to: “The most common case of an initiating ebMS Signal messages is the eb3:PullRequest message sent by an Intermediary to pull message from the endpoint (see case 3b and 4 in section 2.5.2.2]” Add after “... to faulty messages.”: “When these signals are sent by the Intermediairy directly to the Endpoint and therefor don’t need to be routed and no routing info is needed. When the messages are sent by an Endpoint and need to be routed to another endpoint they should contain routing information.” L1048: Reference [to 2.5.2.2] is missing. Remove when accepting proposed change above. L1048: Which ebMS error messages can be sent by the initiating MSH? If they can be initiating messages requirements should also be described here. Now only PullRequest messages are described. L1050: After “.. eb3:UserMessage element” add: “Therefor a WS-A EPR as defined in section 2.7.1 (current section, may need change depending on approval of proposal above) MUST be added to the message.” L1062-1066: Change first part of paragraph up to the sentence starting with “The content of ...” to: “These messages will not contain an eb3:Messaging header unless they are piggybacked on an ebMS message. Therefor a WS-A EPR MUST be added to these messages when not piggybacked on an ebMS message. In the latter case the routing information from the ebMS message is applied. A WS-A replyTo header MAY be added to these messages.” L1064: Here it is stated that non ebMS messsage may contain an WS-A replyTo header. But isn’t that allowed for all messages, including ebMS User and Signal messages? L1077: Isn’t there another possibility that an ebMS User message is a response to a PullRequest? L1086: Add sentence to end of line “An Endpoint MSH MUST use one of these option to ensure that the response message can be routed to its destination.” L1088: Insert at the end of sentence (after "... User messages") " by the PMode configuration of the Responding endpoint and/or by the submitting message Producer" L1088: Why only use the reference parameter from the wsa:ReplyTo ? Line 1092-93 also seems to suggest that other information from the EPR can be used. L1092-1093: Unclear how wsa:To will be derived from wsa:Action in wsa:ReplyTo. L1096: Add after "... eb3:userMessage element" : ", unless the signals are piggybacked to en ebMS User Message." L1096: Change "However, either one these" to "Therefor, when an ebMS Signal message is not piggybacked to an ebMS User Message, either one the following" L1098-1102: Change text to: "the wsa:ReplyTo header was present in the request message. If it contains a reference parameter as defined in section 2.7.1 this reference parameter MUST be used in the response message. When the wsa:ReplyTo header in the request message contains a reachable URI the response message MAY be sent directly to that URI. When the response message is an eb3:Error, the wsa:FaultTo if present MUST take precedence over wsa:ReplyTo." L1107-1123: Unclear to me what the requirements are on the sender of the response message and for which message the requirements On 21/10/2009 00:35, "Sander Fieten" <sander@fieten-it.com> wrote: Below are my comments after reviewing version 0.43 of the part 2 draft. |
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]