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 "Thorsten Schöning (JIRA)" <ji...@apache.org> on 2017/08/25 17:23:00 UTC
[jira] [Comment Edited] (AXIS2-5793)
DBCwithReduceWSDLMemoryParmsTests fails if pathes with spaces are used
[ https://issues.apache.org/jira/browse/AXIS2-5793?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16141919#comment-16141919 ]
Thorsten Schöning edited comment on AXIS2-5793 at 8/25/17 5:22 PM:
-------------------------------------------------------------------
AXIS2-5793: URL.getFile is as bad as File.toURL in case of spaces in paths, because while toURL doesn't rewrite spaces to %20, getFile on the other hand doesn't rewrite %20 back into spaces to use in the file system. During running the tests this lead to modules/webapp creating a wrong directory:
C:\Users\tschoening\Documents\Eclipse\Java\Libs%20Java%203rd\Apache\Axis2\1.7\src\systests\webapp-tests\target\tmp\webapp\WEB-INF\services
vs.
C:\Users\tschoening\Documents\Eclipse\Java\Libs Java 3rd\Apache\Axis2\1.7\src\systests\webapp-tests\target\tmp\webapp\WEB-INF\services
https://stackoverflow.com/a/7700050/2055163
The funny thing is that this path was never used anywhere, it was just created and even if it couldn't be that wasn't checked as well. I checked that using deny permissions in the file system, so that the path couldn't be created in hope I get an exception to find the the place in the code. So I wander if the code in the CTOR is needed at all? The tests all passed while AdminActions created the wrong path. :-)
was (Author: tschoening):
AXIS2-5793: URL.getFile is as bad as File.toURL in case of spaces in paths, because while toURL doesn't rewrite spaces to %20, getFile on the other hand doesn't rewrite %20 back into spaces to use in the file system. During running the tests this lead to modules/webapp creating a wrong directory:
C:\Users\tschoening\Documents\Eclipse\Java\Libs%20Java%203rd\Apache\Axis2\1.7\src\systests\webapp-tests\target\tmp\webapp\WEB-INF\services
vs.
C:\Users\tschoening\Documents\Eclipse\Java\Libs Java 3rd\Apache\Axis2\1.7\src\systests\webapp-tests\target\tmp\webapp\WEB-INF\services
The funny thing is that this path was never used anywhere, it was just created and even if it couldn't be that wasn't checked as well. I checked that using deny permissions in the file system, so that the path couldn't be created in hope I get an exception to find the the place in the code. So I wander if the code in the CTOR is needed at all? The tests all passed while AdminActions created the wrong path. :-)
> DBCwithReduceWSDLMemoryParmsTests fails if pathes with spaces are used
> ----------------------------------------------------------------------
>
> Key: AXIS2-5793
> URL: https://issues.apache.org/jira/browse/AXIS2-5793
> Project: Axis2
> Issue Type: Bug
> Affects Versions: 1.7.3
> Environment: Windows 8.1 x64 Prof., Tomcat 7.0.68, JDK 1.8.0_92 x64
> Reporter: Thorsten Schöning
> Attachments: AXIS2-5793 AdminActions.patch, AXIS2-5793 all others.patch, DescriptionTestUtils2.java.patch, DescriptionTestUtils.java.patch, TEST-org.apache.axis2.jaxws.description.DBCwithReduceWSDLMemoryParmsTests.xml
>
>
> DBCwithReduceWSDLMemoryParmsTests fails when Axis2 is build using a path with spaces:
> {CODE}
> <testcase time="0.017" classname="org.apache.axis2.jaxws.description.DBCwithReduceWSDLMemoryParmsTests" name="testDBCwithReduceWSDLMemoryParms">
> <failure type="junit.framework.AssertionFailedError:">junit.framework.AssertionFailedError: null
> at junit.framework.Assert.fail(Assert.java:47)
> at junit.framework.Assert.assertTrue(Assert.java:20)
> at junit.framework.Assert.assertTrue(Assert.java:27)
> at org.apache.axis2.jaxws.description.DBCwithReduceWSDLMemoryParmsTests.testDBCwithReduceWSDLMemoryParms(DBCwithReduceWSDLMemoryParmsTests.java:140)
> </failure>
> {CODE}
> After enabling logging using log4j the reason is easier to find:
> {CODE}
> 2016-06-25 11:00:36,857 DEBUG org.apache.axis2.jaxws.util.WSDL4JWrapper.<init>: WSDL4JWrapper(Definition, boolean) entry
> 2016-06-25 11:00:36,857 DEBUG org.apache.axis2.wsdl.util.WSDLDefinitionWrapper.<init>: WSDLDefinitionWrapper(Definition,URL,boolean) entry
> 2016-06-25 11:00:36,857 DEBUG org.apache.axis2.wsdl.util.WSDLWrapperReloadImpl.isReloadable: Enter WSDLWrapperReloadImpl.isReloadable(): null
> 2016-06-25 11:00:36,857 DEBUG org.apache.axis2.wsdl.util.WSDLWrapperReloadImpl.getExplicitURI: WSDLWrapperReloadImpl.getExplicitURI(file:/C:/Users/tschoening/Documents/Eclipse/Java Bug 2187/Libs Java 3rd/Apache/Axis2/1.7.3/src/modules/metadata/test-resources/wsdl/BindingNamespaceDefaults.wsdl)
> 2016-06-25 11:00:36,857 DEBUG org.apache.axis2.wsdl.util.WSDLWrapperReloadImpl.isReloadable: WSDLWrapperReloadImpl.isReloaded(): [javax.wsdl.WSDLException] error [WSDLException: faultCode=WSDLWrapperReloadImpl : : java.net.URISyntaxException: Illegal character in path at index 48: file:/C:/Users/tschoening/Documents/Eclipse/Java Bug 2187/Libs Java 3rd/Apache/Axis2/1.7.3/src/modules/metadata/test-resources/wsdl/BindingNamespaceDefaults.wsdl]
> javax.wsdl.WSDLException: WSDLException: faultCode=WSDLWrapperReloadImpl : : java.net.URISyntaxException: Illegal character in path at index 48: file:/C:/Users/tschoening/Documents/Eclipse/Java Bug 2187/Libs Java 3rd/Apache/Axis2/1.7.3/src/modules/metadata/test-resources/wsdl/BindingNamespaceDefaults.wsdl
> at org.apache.axis2.wsdl.util.WSDLWrapperReloadImpl.getExplicitURI(WSDLWrapperReloadImpl.java:1382)
> {CODE}
> The problem is in DescriptionTestUtils.getWSDLURL:
> {CODE}
> wsdlURL = new File(urlString).getAbsoluteFile().toURL();
> {CODE}
> toURL is kown to fail in situations like these:
> {QUOTE}
> Deprecated. This method does not automatically escape characters that are illegal in URLs. It is recommended that new code convert an abstract pathname into a URL by first converting it into a URI, via the toURI method, and then converting the URI into a URL via the URI.toURL method.
> {QUOTE}
> https://docs.oracle.com/javase/7/docs/api/java/io/File.html#toURL()
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscribe@axis.apache.org
For additional commands, e-mail: java-dev-help@axis.apache.org