From db45b5640ab577057e630b1c14bd537517aa5a29 Mon Sep 17 00:00:00 2001
From: Per Wendel <per.i.wendel@gmail.com>
Date: Tue, 6 Mar 2018 11:42:19 +0100
Subject: [PATCH] Fix for #981 (#987)

---
 pom.xml                                       |  2 +-
 .../spark/resource/ClassPathResource.java     | 27 ++++++++++---------
 2 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/pom.xml b/pom.xml
index f8e0a98a15..5ed76d0065 100644
--- a/pom.xml
+++ b/pom.xml
@@ -9,7 +9,7 @@
     <groupId>com.sparkjava</groupId>
     <artifactId>spark-core</artifactId>
     <packaging>bundle</packaging>
-    <version>3.0.0-SNAPSHOT</version>
+    <version>2.7.2-SNAPSHOT</version>
     <name>Spark</name>
     <description>A Sinatra inspired java web framework</description>
     <url>http://www.sparkjava.com</url>
diff --git a/src/main/java/spark/resource/ClassPathResource.java b/src/main/java/spark/resource/ClassPathResource.java
index 70bccc0918..dd8ba97770 100644
--- a/src/main/java/spark/resource/ClassPathResource.java
+++ b/src/main/java/spark/resource/ClassPathResource.java
@@ -74,14 +74,22 @@ public ClassPathResource(String path) {
      */
     public ClassPathResource(String path, ClassLoader classLoader) {
         Assert.notNull(path, "Path must not be null");
+        Assert.state(doesNotContainFileColon(path), "Path must not contain 'file:'");
+
         String pathToUse = StringUtils.cleanPath(path);
+
         if (pathToUse.startsWith("/")) {
             pathToUse = pathToUse.substring(1);
         }
+
         this.path = pathToUse;
         this.classLoader = (classLoader != null ? classLoader : ClassUtils.getDefaultClassLoader());
     }
 
+    private static boolean doesNotContainFileColon(String path) {
+        return !path.contains("file:");
+    }
+
     /**
      * Create a new ClassPathResource with optional ClassLoader and Class.
      * Only for internal usage.
@@ -108,10 +116,9 @@ public final String getPath() {
     /**
      * This implementation checks for the resolution of a resource URL.
      *
+     * @return if exists.
      * @see java.lang.ClassLoader#getResource(String)
      * @see java.lang.Class#getResource(String)
-     *
-     * @return if exists.
      */
     @Override
     public boolean exists() {
@@ -127,10 +134,9 @@ public boolean exists() {
     /**
      * This implementation opens an InputStream for the given class path resource.
      *
+     * @return the input stream.
      * @see java.lang.ClassLoader#getResourceAsStream(String)
      * @see java.lang.Class#getResourceAsStream(String)
-     *
-     * @return the input stream.
      */
     @Override
     public InputStream getInputStream() throws IOException {
@@ -149,10 +155,9 @@ public InputStream getInputStream() throws IOException {
     /**
      * This implementation returns a URL for the underlying class path resource.
      *
+     * @return the url.
      * @see java.lang.ClassLoader#getResource(String)
      * @see java.lang.Class#getResource(String)
-     *
-     * @return the url.
      */
     @Override
     public URL getURL() throws IOException {
@@ -172,9 +177,8 @@ public URL getURL() throws IOException {
      * This implementation creates a ClassPathResource, applying the given path
      * relative to the path of the underlying resource of this descriptor.
      *
-     * @see spark.utils.StringUtils#applyRelativePath(String, String)
-     *
      * @return the resource.
+     * @see spark.utils.StringUtils#applyRelativePath(String, String)
      */
     @Override
     public Resource createRelative(String relativePath) {
@@ -186,9 +190,8 @@ public Resource createRelative(String relativePath) {
      * This implementation returns the name of the file that this class path
      * resource refers to.
      *
-     * @see spark.utils.StringUtils#getFilename(String)
-     *
      * @return the file name.
+     * @see spark.utils.StringUtils#getFilename(String)
      */
     @Override
     public String getFilename() {
@@ -233,8 +236,8 @@ public boolean equals(Object obj) {
             ClassLoader otherLoader = otherRes.classLoader;
 
             return (this.path.equals(otherRes.path) &&
-                    thisLoader.equals(otherLoader) &&
-                    this.clazz.equals(otherRes.clazz));
+                thisLoader.equals(otherLoader) &&
+                this.clazz.equals(otherRes.clazz));
         }
         return false;
     }