On Thu March 5 2009 12:52:21 pm [email protected] wrote:
>          CachingXmlEventWriter cache = null;
>          
> -        if (shouldValidate(message) && !isRequestor(message)) {
> -            //need to cache the events in case validation fails
> +        // need to cache the events in case validation fails or buffering
> is enabled +        if (shouldValidate(message) && !isRequestor(message)
> +            || SystemUtils.isBufferingEnabled()) {
>              cache = new CachingXmlEventWriter();
>              try {

This definitely shouldn't be based on a system property.   It should be 
something like:

message.getContextualProperty("org.apache.cxf.buffer.output")

or similar so it can be configured on a per-endpoint basis.


-- 
Daniel Kulp
[email protected]
http://www.dankulp.com/blog

Reply via email to