8183 2009-11-04 18:35:36 +0000 Transfer: be clear on fault for bad representation 2010-03-17 11:31:35 +0000 1 1 1 Unclassified WS-Resource Access Transfer FPWD PC Windows XP CLOSED REMIND P2 normal --- 1 dug dug ram.jeyaraman public-ws-resource-access-notifications oldest_to_newest 29311 0 dug 2009-11-04 18:35:36 +0000 Transfer.Put() says: -- Implementations MAY use the fault code wst:InvalidRepresentation if the presented representation is invalid for the target resource. The replacement representation could be considered to be invalid if it does not conform to the schema(s) for the target resource or otherwise violates some cardinality or type constraint. If an implementation detects that the presented representation is invalid it MUST generate a wst:InvalidRepresentation fault. -- The MAY in the first sentence is inconsistent with the MUST at the end. Also, Create says: -- In addition to the standard fault codes defined in WS-Addressing, implementations MAY use the fault code wst:InvalidRepresentation if the presented representation is invalid for the target resource. -- Why mention WSA faults? that's just odd. Why MAY? For consistency, and interop, we should make this a MUST. Proposal: Replace first para with: Implementations MUST use the fault code wst:InvalidRepresentation if the presented representation is invalid for the target resource. The replacement representation could be considered to be invalid if it does not conform to the schema(s) for the target resource or otherwise violates some cardinality or type constraint. (s/MAY/MUST/ and remove last sentence) And in Create replace that sentence with: An implementation MUST use the fault code wst:InvalidRepresentation if the presented representation is invalid for the target resource. 29312 1 dug 2009-11-04 18:39:22 +0000 just noticed that Delete says: -- Implementations MAY respond with a fault message using the standard fault codes defined in WS-Addressing (e.g., wsa:ActionNotSupported). -- too. Amended proposal: remove this sentence too. We don't need to repeat the WSA spec. 30310 2 ram.jeyaraman 2010-01-02 18:24:46 +0000 I like to suggest a slightly modified proposal below. This allows the implementation to decide whether or not to do schema validation. Transfer.Put(): From: Implementations MAY use the fault code wst:InvalidRepresentation if the presented representation is invalid for the target resource. To: Implementations MUST use the fault code wst:InvalidRepresentation if they detect that the presented representation is invalid for the target resource. Transfer.Create(): From: In addition to the standard fault codes defined in WS-Addressing, implementations MAY use the fault code wst:InvalidRepresentation if the presented representation is invalid for the target resource. To: An implementation MUST use the fault code wst:InvalidRepresentation if it detects that the presented representation is invalid for the target resource. Transfer.Delete : Remove: Implementations MAY respond with a fault message using the standard fault codes defined in WS-Addressing (e.g., wsa:ActionNotSupported). 30550 3 dug 2010-01-05 21:42:52 +0000 http://lists.w3.org/Archives/Public/public-ws-resource-access/2010Jan/0025.html 30976 4 bob 2010-01-12 15:17:05 +0000 2010-01-05 resolved with: <Ram> If an implementation that performs schema validation on a presented representation detects that the presented representation is invalid for the target resource, then the implementation MUST use the fault code wst:InvalidRepresentation. Proposal: modify 1st sentence of Put text, and Create text, with Ram's text above. Remove last sentence of Put para. Remove the WSA text from delete.