This is an automated email from the ASF dual-hosted git repository.
remm pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new a5e0e42101 Fix missing IllegalArgumentException
a5e0e42101 is described below
commit a5e0e42101e3c1e50e58510775992a0c5167a0f3
Author: remm <r...@apache.org>
AuthorDate: Tue Apr 18 17:05:04 2023 +0200
Fix missing IllegalArgumentException
Regression when the Tomcat code was converted to using URI instead of
URL. Only a few items really need it, but be somewhat cautious. Other
instances of use seem fine.
---
java/org/apache/catalina/connector/Response.java | 2 +-
java/org/apache/catalina/core/NamingContextListener.java | 4 ++--
java/org/apache/catalina/startup/ContextConfig.java | 4 ++--
.../catalina/tribes/membership/cloud/AbstractStreamProvider.java | 2 +-
java/org/apache/catalina/valves/ProxyErrorReportValve.java | 2 +-
.../org/apache/catalina/webresources/AbstractArchiveResource.java | 2 +-
java/org/apache/catalina/webresources/JarResourceRoot.java | 2 +-
webapps/docs/changelog.xml | 8 ++++++++
8 files changed, 17 insertions(+), 9 deletions(-)
diff --git a/java/org/apache/catalina/connector/Response.java
b/java/org/apache/catalina/connector/Response.java
index 496790900f..57de133eb8 100644
--- a/java/org/apache/catalina/connector/Response.java
+++ b/java/org/apache/catalina/connector/Response.java
@@ -1533,7 +1533,7 @@ public class Response implements HttpServletResponse {
try {
URI uri = new URI(location);
url = uri.toURL();
- } catch (MalformedURLException | URISyntaxException e) {
+ } catch (MalformedURLException | URISyntaxException |
IllegalArgumentException e) {
return false;
}
diff --git a/java/org/apache/catalina/core/NamingContextListener.java
b/java/org/apache/catalina/core/NamingContextListener.java
index d8b5c3cdbb..deff7e8252 100644
--- a/java/org/apache/catalina/core/NamingContextListener.java
+++ b/java/org/apache/catalina/core/NamingContextListener.java
@@ -831,7 +831,7 @@ public class NamingContextListener implements
LifecycleListener, ContainerListen
try {
URI wsdlURI = new URI(service.getWsdlfile());
wsdlURL = wsdlURI.toURL();
- } catch (MalformedURLException | URISyntaxException e) {
+ } catch (MalformedURLException | URISyntaxException |
IllegalArgumentException e) {
// Ignore and carry on
}
if (wsdlURL == null) {
@@ -862,7 +862,7 @@ public class NamingContextListener implements
LifecycleListener, ContainerListen
try {
URI jaxrpcURI = new URI(service.getJaxrpcmappingfile());
jaxrpcURL = jaxrpcURI.toURL();
- } catch (MalformedURLException | URISyntaxException e) {
+ } catch (MalformedURLException | URISyntaxException |
IllegalArgumentException e) {
// Ignore and carry on
}
if (jaxrpcURL == null) {
diff --git a/java/org/apache/catalina/startup/ContextConfig.java
b/java/org/apache/catalina/startup/ContextConfig.java
index 95a6ef4ce8..5d463f5383 100644
--- a/java/org/apache/catalina/startup/ContextConfig.java
+++ b/java/org/apache/catalina/startup/ContextConfig.java
@@ -1685,7 +1685,7 @@ public class ContextConfig implements LifecycleListener {
URL url = uri.toURL();
uc = url.openConnection();
globalTimeStamp = uc.getLastModified();
- } catch (IOException | URISyntaxException e) {
+ } catch (IOException | URISyntaxException |
IllegalArgumentException e) {
globalTimeStamp = -1;
} finally {
if (uc != null) {
@@ -1706,7 +1706,7 @@ public class ContextConfig implements LifecycleListener {
URL url = uri.toURL();
uc = url.openConnection();
hostTimeStamp = uc.getLastModified();
- } catch (IOException | URISyntaxException e) {
+ } catch (IOException | URISyntaxException |
IllegalArgumentException e) {
hostTimeStamp = -1;
} finally {
if (uc != null) {
diff --git
a/java/org/apache/catalina/tribes/membership/cloud/AbstractStreamProvider.java
b/java/org/apache/catalina/tribes/membership/cloud/AbstractStreamProvider.java
index 2361169710..5996113496 100644
---
a/java/org/apache/catalina/tribes/membership/cloud/AbstractStreamProvider.java
+++
b/java/org/apache/catalina/tribes/membership/cloud/AbstractStreamProvider.java
@@ -81,7 +81,7 @@ public abstract class AbstractStreamProvider implements
StreamProvider {
URLConnection connection;
try {
connection = new URI(url).toURL().openConnection();
- } catch (URISyntaxException e) {
+ } catch (URISyntaxException | IllegalArgumentException e) {
// Not ideal but consistent with API
throw new IOException(e);
}
diff --git a/java/org/apache/catalina/valves/ProxyErrorReportValve.java
b/java/org/apache/catalina/valves/ProxyErrorReportValve.java
index 1b59a210fe..7220ffabf0 100644
--- a/java/org/apache/catalina/valves/ProxyErrorReportValve.java
+++ b/java/org/apache/catalina/valves/ProxyErrorReportValve.java
@@ -219,7 +219,7 @@ public class ProxyErrorReportValve extends ErrorReportValve
{
OutputStream outputStream = response.getOutputStream();
InputStream inputStream = url.openStream();
IOUtils.copy(inputStream, outputStream);
- } catch (URISyntaxException | IOException e) {
+ } catch (URISyntaxException | IOException |
IllegalArgumentException e) {
if (log.isDebugEnabled()) {
log.debug("Proxy error to " + urlString, e);
}
diff --git a/java/org/apache/catalina/webresources/AbstractArchiveResource.java
b/java/org/apache/catalina/webresources/AbstractArchiveResource.java
index 982d676f12..d4f2e1c4a1 100644
--- a/java/org/apache/catalina/webresources/AbstractArchiveResource.java
+++ b/java/org/apache/catalina/webresources/AbstractArchiveResource.java
@@ -144,7 +144,7 @@ public abstract class AbstractArchiveResource extends
AbstractResource {
String url = baseUrl + URLEncoder.DEFAULT.encode(resource.getName(),
StandardCharsets.UTF_8);
try {
return new URI(url).toURL();
- } catch (MalformedURLException | URISyntaxException e) {
+ } catch (MalformedURLException | URISyntaxException |
IllegalArgumentException e) {
if (getLog().isDebugEnabled()) {
getLog().debug(sm.getString("fileResource.getUrlFail", url),
e);
}
diff --git a/java/org/apache/catalina/webresources/JarResourceRoot.java
b/java/org/apache/catalina/webresources/JarResourceRoot.java
index 27686edf76..cdd6e1b1fa 100644
--- a/java/org/apache/catalina/webresources/JarResourceRoot.java
+++ b/java/org/apache/catalina/webresources/JarResourceRoot.java
@@ -125,7 +125,7 @@ public class JarResourceRoot extends AbstractResource {
String url = baseUrl + "!/";
try {
return new URI(url).toURL();
- } catch (MalformedURLException | URISyntaxException e) {
+ } catch (MalformedURLException | URISyntaxException |
IllegalArgumentException e) {
if (log.isDebugEnabled()) {
log.debug(sm.getString("fileResource.getUrlFail", url), e);
}
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 06f98d2625..567994ec26 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -105,6 +105,14 @@
issues do not "pop up" wrt. others).
-->
<section name="Tomcat 9.0.75 (remm)" rtext="in development">
+ <subsection name="Catalina">
+ <changelog>
+ <fix>
+ <bug>66567</bug>: Fix missing <code>IllegalArgumentException</code>
+ after the Tomcat code was converted to using URI instead of URL. (remm)
+ </fix>
+ </changelog>
+ </subsection>
</section>
<section name="Tomcat 9.0.74 (remm)" rtext="release in progress">
<subsection name="Catalina">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
