Great! :) ..

Cheers,
Anjana.

On Fri, Jul 13, 2012 at 9:51 PM, Pradeep Fernando <[email protected]> wrote:

> Hi,
>
> Had a look. This was broken due to a harmless commit. :)
>
> within the carbon-code, we are reading property values, like
>
> private static final String defaultInitialContextFactory =
>                 CarbonUtils.getServerConfiguration().getFirstProperty(
>                         "JNDI.DefaultInitialContextFactory");
>
> As you can see we are not trimming/formatting/checking the returned value.
> So if we change the format of the config files, then we are in trouble. :)
>
> Class.forName methods, axis2 Module loading failed due to this. However it
> was a nice debugging session. :)
>
> we should correct the code.
>
>
> thanks,
> --Pradeep
>



-- 
*Anjana Fernando*
Associate Technical Lead
WSO2 Inc. | http://wso2.com
lean . enterprise . middleware
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to