Merge pull request #2547 from armanbilge/topic/js-exceptions

Catch a couple more javascript exceptions
typelevel · Aug 7, 2021 · ad9cea3 · ad9cea3
2 parents 730d3c5 + 035a5f7
commit ad9cea3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/io/js/src/main/scala/fs2/io/net/NetException.scala b/io/js/src/main/scala/fs2/io/net/NetException.scala
@@ -87,7 +87,8 @@ private class JavaScriptUnknownException(cause: js.JavaScriptException)
 object UnknownHostException {
   private[io] def unapply(cause: js.JavaScriptException): Option[UnknownHostException] =
     cause match {
-      case js.JavaScriptException(error: js.Error) if error.message.contains("ENOTFOUND") =>
+      case js.JavaScriptException(error: js.Error)
+          if error.message.contains("ENOTFOUND") || error.message.contains("EAI_AGAIN") =>
         Some(new JavaScriptUnknownException(cause))
       case _ => None
     }

diff --git a/io/js/src/main/scala/fs2/io/net/tls/SSLException.scala b/io/js/src/main/scala/fs2/io/net/tls/SSLException.scala
@@ -33,7 +33,9 @@ private class JavaScriptSSLException(cause: js.JavaScriptException)
 object SSLException {
   private[io] def unapply(cause: js.JavaScriptException): Option[SSLException] = cause match {
     case js.JavaScriptException(error: js.Error)
-        if error.message.contains("TLS") || error.message.contains("SSL") =>
+        if error.message.contains("TLS") || error.message.contains(
+          "SSL"
+        ) || error.message.toLowerCase.contains("certificate") =>
       Some(new JavaScriptSSLException(cause))
     case _ => None
   }