Skip to content

Commit ad9cea3

Browse files
authored
Merge pull request #2547 from armanbilge/topic/js-exceptions
Catch a couple more javascript exceptions
2 parents 730d3c5 + 035a5f7 commit ad9cea3

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

io/js/src/main/scala/fs2/io/net/NetException.scala

+2-1
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,8 @@ private class JavaScriptUnknownException(cause: js.JavaScriptException)
8787
object UnknownHostException {
8888
private[io] def unapply(cause: js.JavaScriptException): Option[UnknownHostException] =
8989
cause match {
90-
case js.JavaScriptException(error: js.Error) if error.message.contains("ENOTFOUND") =>
90+
case js.JavaScriptException(error: js.Error)
91+
if error.message.contains("ENOTFOUND") || error.message.contains("EAI_AGAIN") =>
9192
Some(new JavaScriptUnknownException(cause))
9293
case _ => None
9394
}

io/js/src/main/scala/fs2/io/net/tls/SSLException.scala

+3-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,9 @@ private class JavaScriptSSLException(cause: js.JavaScriptException)
3333
object SSLException {
3434
private[io] def unapply(cause: js.JavaScriptException): Option[SSLException] = cause match {
3535
case js.JavaScriptException(error: js.Error)
36-
if error.message.contains("TLS") || error.message.contains("SSL") =>
36+
if error.message.contains("TLS") || error.message.contains(
37+
"SSL"
38+
) || error.message.toLowerCase.contains("certificate") =>
3739
Some(new JavaScriptSSLException(cause))
3840
case _ => None
3941
}

0 commit comments

Comments
 (0)