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

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


Mime
View raw message