james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Burrell Donkin (JIRA)" <server-...@james.apache.org>
Subject [jira] Commented: (JSIEVE-5) You'd better change the way you load sieveConfig.xml
Date Thu, 09 Aug 2007 19:53:42 GMT

    [ 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

   return Thread.currentThread().getContextClassLoader().getResourceAsStream(getConfigName());
  return ClassLoader.getSystemResourceAsStream(getConfigName());

might work well enough for now.


> 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: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org

View raw message