joinr's fix broke parsing urls with schemes not registered with the jvm.

src/tech/v3/io/url.clj

@@ -8,7 +8,12 @@
 
 
 (defn maybe-url [^String url]
-  (try (java.net.URL. url) (catch Exception e nil)))
+  (re-find #"[A-Za-z0-9]+:\/\/" url)
+  ;;Leaving here to make a point.  The below fails for any url
+  ;;that doesn't have a scheme registered with the central java url
+  ;;system - such as s3 or azb.
+  #_(try (java.net.URL. url)
+         (catch Exception e nil)))
 
 
 (defn parse-url-arguments

test/tech/v3/io/url_test.clj

@@ -9,6 +9,12 @@
   (is (not (nil? (url/url->parts "s3://a/b/c"))))
   (is (not (nil? (url/url->parts "makeitup://a/b/c")))))
 
+
+(deftest valid-url-test
+  (is (url/url? "s3://a/b/c"))
+  (is (url/url? "https://a/b/c"))
+  (is (not (url/url? "C:\\\\Users\\\\joinr\\\\data\\\\yellow_tripdata_2016-01.csv"))))
+
 (deftest windows-paths-are-not-urls
   (let [p "C:\\Users\\joinr\\data\\yellow_tripdata_2016-01.csv"]
     (is (not (boolean (url/url? p))))))