[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [Elist Home]
Subject: [wsrp-wsia] WSRP 0.90
Section: 1.2.4 Page/Line: 39 Requestedby: Andre Kramer Old text: Proposed text: A consumer should only intitate one performBlockingInteraction or performInteraction at a time, on behalf of a User. Reasoning: Natural for a portal, but not clear this is always required (for non-blocking interactions). Section: 5.1.2 Page/Line: move to 3.3 Requestedby: Andre Kramer Old text: Proposed text: Reasoning: Get our extensibility mechanism definition out of the flow of service description Section: 5.1.11 PortletDescription.userContextStoredInSession Page/Line: Requestedby: Andre Kramer Old text: Proposed text: The session referred to here is a Portlet session or the HTTP session that scopes the Portlet session if one exists. Reasoning: If requiresInitCookie <> "none" then a per-user HTTP session is active for the user. We may wish to re-visit whether templatesStoredInSession is scoped in the same way (as an optimization). Section: 5.1.17 ServiceDescription Page/Line: new member Requestedby: Andre Kramer Old text: Proposed text: [O] string[] locales Reasoning: What locales the service description is available in is not advertised. Unlike PortalDescription.markupTypes[].locales[], there is no clear way to work out what desiredLocales[] to request with getServiceDescription(). Remember, getServiceDescription() typically needs to be called twice anyway so why not help seed desiredLocales? Section: 5.1.15 PropertyDescription Page/Line: 25 Requestedby: Andre Kramer Old text: Proposed text: [O] string propertyUsage propertyUsage: Describes whether the property is read-only (propertyUsage == "readOnly") or optional (propertyUsage == "optional"); with the producer supplying a default value for optional properties. Reasoning: Property types are overloaded for Portlet and Registration properties and we have no way to indicate which are optional and which are read-only (especially important for registration). Not clear if consumer can currently ignore registration properties it does not understand. Section: 5.1.18 Page/Line: 27 Requestedby: Andre Kramer Old text: userContextID: This key ... Proposed text: replace key with ID Reasoning: The ID description states that an ID is unlikely to be used as a key. However, this ID is likely to be used as a Key. Section: 6.1.5 Page/Line: 27 Requestedby: Andre Kramer Old text: secureDefaultTemplate field MAY provide a first level override ... Proposed text: secureDefaultTemplate field MUST be used as the default for all secure URLs that the portlet writes. If not present a fault is thrown by the producer if the Portlet attempts to write a secure URL. Reasoning: Dangerous to fall back to non-secure if consumer does not supply any reliable way to secure requests. So we do need the RequiresSecureCommunications fault? Section: 6.1.9 Page/Line: 35/38 Requestedby: Andre Kramer Old text: Proposed text: [O] Markup markup where: <complexType name="Markup"> <sequence> <any namespace="##other" /> </sequence> </complexType> <element name="markup" type="types:Markup" minOccurs="0"/> Reasoning: We cover non-XML text with: <element name="markupString" type="xsd:string" minOccurs="0" /> and binary markup with: <element name="markupBinary" type="xsd:base64Binary" minOccurs="0" /> This "any" allows XML to be inlined naturally. Section: 6.1.13 Page/Line: 38/35 Requestedby: Andre Kramer Old text: Proposed text: [O] boolean rewriteRedirectURL rewriteRedirectURL: if redirectURL is non-null then this boolean directs the consumer to apply consumer URL re-writing if true (default is false). The redirectURL value MUST be a Resource URL (10.2.1.1.3). Section: 6.5 releaseSession Page/Line: 43/ Requestedby: Andre Kramer Old text: Proposed text: SessionIDs may be set to null in releaseSession() calls. In which case, the consumer is signalling that any underlying HTTP session is no longer to be considered active. The consumer may want to release any session identifiers it caused to be established in the scope of the HTTP session, independently of the HTTP session (but is likely to be supplying portletInstanceKeys). Section: 7.9 Page/Line: 52/12 Requestedby: Andre Kramer Old text: Proposed text: The consumer is strongly encouraged to re-try deregistrations that failed due to network problems (at a later time) or to seek administrative intervention. Section: 8 Page/Line: 52/27 Requestedby: Andre Kramer Old text: Proposed text: Any producer that lazily clones portlets on performBlockingInteraction() or performInteraction(), via the CloneOnWrite mechanism (6.3.3) SHOULD/MUST support the releasePortlet() operation. Reasoning: Cross dependency of optional and required factors. Section: 8.1.4 PortletPropertyDescriptionResponse Page/Line: 53 Requestedby: Andre Kramer Old text: [O] ModelDescription Proposed text: Make minOccurs="0" in the wsdl as well. Section: 8.5 Page/Line: 55/46 Requestedby: Andre Kramer Old text: Proposed text: A consumer SHOULD perform a getPortletProperties() after a setPortletProperties() on a Portlet before displaying that Portlet's properties to a user. Section: 10.2.1.1.4.2 Page/Line: 61/42 Requestedby: Andre Kramer Old text: Proposed text: The default value for wsrp-rewriteResource is true. Reason: Default could be false, but still seems somewhat unclear how to use URL templates for resource writing. regards, Andre
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [Elist Home]
Powered by eList eXpress LLC