[ 
https://issues.apache.org/jira/browse/JSIEVE-5?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12518827
 ] 

Robert Burrell Donkin commented on JSIEVE-5:
--------------------------------------------

...Grrrr...Don't get me started on J2EE classloaders...

Really wants to be refactored so that a suitable interface is injected but

try
{
   return 
Thread.currentThread().getContextClassLoader().getResourceAsStream(getConfigName());
}
catch
{
  return ClassLoader.getSystemResourceAsStream(getConfigName());
} 

might work well enough for now.

Opinions?

> You'd better change the way you load sieveConfig.xml
> ----------------------------------------------------
>
>                 Key: JSIEVE-5
>                 URL: https://issues.apache.org/jira/browse/JSIEVE-5
>             Project: jSieve
>          Issue Type: Improvement
>            Reporter: Vladimir
>
> I've placed sieveConfig.xml in jsieve's jar (I think that might be much 
> better if you use jsieve as a library).
> You'd better change in ConfigurationManager method: getConfigStream
> this: return ClassLoader.getSystemResourceAsStream(getConfigName());
> with this:
> return 
> Thread.currentThread().getContextClassLoader().getResourceAsStream(getConfigName());
> ClassLoader doesn't load config if you use jsieve as a library

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to