[
http://jira.jboss.com/jira/browse/JBPORTAL-1784?page=comments#action_1238... ]
Dave Rowe commented on JBPORTAL-1784:
-------------------------------------
Hi Chris. While I agree with you that the specification states the two fields are
mutually exclusive, I would say the specification still allows for the existence of the
field with empty content. The only time the specification states the field should not
exist at all is in the case where 'useCachedMarkup' is 'true'. Otherwise,
I would interpret the spec to allow for the field to exist with empty content. This is
why I suggest the zero length check. The specification doesn't state the fields
themselves are mutually exclusive, only that the markupString field is "mutually
exclusive with the returning of the markup of the markupBinary field".
in summary, I would say it's a producer error to send a _value_ for both markupString
and markupBinary.
WSRP - Markup String / Binary handling - fail correctly
-------------------------------------------------------
Key: JBPORTAL-1784
URL:
http://jira.jboss.com/jira/browse/JBPORTAL-1784
Project: JBoss Portal
Issue Type: Bug
Security Level: Public(Everyone can see)
Components: Portal WSRP
Affects Versions: 2.6.2 Final
Environment: Oracle OC4J 10.1.3.3.0, Jboss 2.6.2
Reporter: Dave Rowe
Assigned To: Chris Laprun
When JBoss checks the Markup response, it only checks if the byte array is not null. It
needs an additional check to validate that the length is greater than zero.
This is in org.jboss.portal.wsrp.consumer.RenderHandler (line 102, as of rev. 7792). Add
in an additional check that binary.length > 0.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira