axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "George Stanchev" <Gstanc...@serena.com>
Subject small problem with FileSystemConfigurator
Date Wed, 06 Dec 2006 21:27:50 GMT
Hi,
 
I checked both the axis2 1.1 tag and the trunk. This file: 
 
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.
 
Let me know if you need a JIRA opened for this.
 
Thanks,
 
George Stanchev

**********************************************************************
This email and any files transmitted with it are confidential and intended solely for the
use of the individual or entity to whom they are addressed. Any unauthorized review, use,
disclosure or distribution is prohibited. If you are not the intended recipient, please contact
the sender by reply e-mail and destroy all copies of the original message.


Mime
View raw message