You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by "Deepal Jayasinghe (JIRA)" <ji...@apache.org> on 2007/02/05 07:58:05 UTC
[jira] Resolved: (AXIS2-1839) FileSystemConfigurator doesn't throw
AxisFault when failed to find a user supplied repo or axis2xml location
[ https://issues.apache.org/jira/browse/AXIS2-1839?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Deepal Jayasinghe resolved AXIS2-1839.
--------------------------------------
Resolution: Fixed
fixed in the current code base
> FileSystemConfigurator doesn't throw AxisFault when failed to find a user supplied repo or axis2xml location
> ------------------------------------------------------------------------------------------------------------
>
> Key: AXIS2-1839
> URL: https://issues.apache.org/jira/browse/AXIS2-1839
> Project: Axis 2.0 (Axis2)
> Issue Type: Bug
> Components: kernel
> Affects Versions: 1.2, 1.1, nightly
> Environment: any
> Reporter: George Stanchev
> Assigned To: Deepal Jayasinghe
>
> org.apache.axis2.deployment.FileSystemConfigurator.FileSystemConfigurator(String, String)
>
> has a bug in it right there where the repoLocation is determined if it is valid or not (lines 57-69):
>
> try {
> if (repoLocation != null) {
> File repo = new File(repoLocation);
> if (repo.exists()) {
> // ok, save it if so
> this.repoLocation = repo.getAbsolutePath();
> }
> }
> } catch (Exception e) {
> log.info("Couldn't find repository location '" +
> repoLocation + "'");
> this.repoLocation = null;
> }
>
> So if the repo path is invalid, the repo.exists() will be false and
> the block will complete gracefully without logging the condition (that
> repo location does not exist). The java.io.File.File(String) constructor
> will throw an exception only if the argument is null, which in our case
> obviously it is not. So that catch() block is a bit pointless unless it is
> intended for the SecurityException possibly thrown by File.getAbsolutePath()
> but then the message is wrong.
>
> FileSystemConfigurator is used by createConfigurationContextFromFileSystem
> method which can throw AxisFaults. I think FileSystemConfigurator should
> throw an exception if the repoLocation or axis2xlm are not found on the system
> instead of merely logging it and falling through the default configuration load.
> Otherwise the problem will snowball later on when the default axis2.xml is loaded,
> modules fail to engage etc and make the user wonder what is going on, like it was
> in my case.
>
> At the very least, besides fixing this problem, log it with error() severity. info() is
> too low for supplied-but-not-found configuration loading.
--
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: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org