Hi Scott,

On Tue, Oct 1, 2013 at 3:55 AM, Scott Came <[email protected]> wrote:

>  I have noticed that the Result message (i.e., the String produced by
> org.wso2.balana.ctx.AbstractResult.encode()) does not appear to qualify
> names with namespaces, as required by the XACML 3.0 spec.
>

AFAIK,  Result element is not needed to be qualify with namespace, if
Response element has been already qualified. If i have missed any thing,
Please let us know....

> ****
>
> ** **
>
> I was looking at class org.wso2.balana.ctx.xacml3.Result.encode() and was
> curious why the output XML is built up as a String, rather than using the
> native DOM capabilities, or similar capabilities in XML open source
> libraries?
>

We do not think to add more complexity here. It is returned the String
value.... then it is needed to build a DOM and convert to String... Do you
think, using DOM is faster than string manipulation?

> ****
>
> ** **
>
> I notice the same issue applies to
> org.wso2.balana.ctx.ResponseCtx.encode(), which actually has some
> commented-out code that would add the namespace declaration…
>

Yes actually it has been commented which is wrong...  this is fixed now.

> ****
>
> ** **
>
> Was there a decision made at some point not to have the response message
> conform to the spec?
>

No.  i guess, this is a mistake.  namespace was there in earlier trunk
revisions.  Thanks for pointing this out.


Thanks,
Asela.

> ****
>
> ** **
>
> Thanks.****
>
> --Scott ****
>
> _______________________________________________
> Dev mailing list
> [email protected]
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>


-- 
Thanks & Regards,
Asela

ATL
Mobile : +94 777 625 933
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to