default on-error-retry-fn? now considers java.net.http.HttpTimeoutException. v0.1.0

Author: bortexz

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change Log
 
+## 0.1.0 - 2022-10-15
+### Changed
+- default `on-error-retry-fn?` now also considers `java.net.http.HttpTimeoutException`
+
 ## 0.0.2 - 2022-09-30
 ### Changed
 - `reconnector` docstring now includes `get-url` and `get-opts` docs. No functional changes.

README.md

@@ -16,12 +16,12 @@ JDK11 Clojure WebSocket client with core.async API
 ## Install
 ### **Leiningen/Boot**
 ```clojure
-[io.github.bortexz/resocket "0.0.2"]
+[io.github.bortexz/resocket "0.1.0"]
 ```
 
 ### **Clojure CLI/deps.edn**
 ```clojure
-io.github.bortexz/resocket {:mvn/version "0.0.2"}
+io.github.bortexz/resocket {:mvn/version "0.1.0"}
 ```
 
 ## Usage
@@ -32,6 +32,8 @@ See the [docs](https://cljdoc.org/d/io.github.bortexz/resocket) for more detaile
 
 Connection:
 ```Clojure
+(require '[bortexz.resocket :as resocket])
+
 (let [{:keys [input output closed error]} (a/<!! (resocket/connection "ws://<service>" {}))]
     (when-not error
       (a/<!! input) ;; Take new messages

build.clj

@@ -3,7 +3,7 @@
   (:require [org.corfield.build :as bb]))
 
 (def lib 'io.github.bortexz/resocket)
-(def version "0.0.2")
+(def version "0.1.0")
 
 (defn- gha-output
   [k v]

src/bortexz/resocket.clj

@@ -272,13 +272,17 @@
    - `retry-ms-fn` unary fn called with the current attempt at reconnecting, starting at 1. It must return a numer of 
      milliseconds to wait before retrying a new connection, or nil to close the reconnector. Defaults to 5000 (5 secs).
    - `on-error-retry-fn?` unary fn called with error when creating a new connection. Must return true to start a retry
-     timeout to retry, otherwise the reconnector will be closed. Note: It is called with the original Exception unwrapped,
-     which will probably be an ExecutionException from the CompletableFuture returned by creating a new Websocket in JDK11.
-     You might want to get the wrapped exception for more details `(ex-cause <error>)`. Defaults to returning true 
-     when the wrapped exception is a java.net.ConnectException, false otherwise."
+     timeout to retry, otherwise the reconnector will be closed. 
+     Note: It is called with the original Exception unwrapped, which will be an ExecutionException from the 
+     CompletableFuture returned by creating a new Websocket in JDK11. You might want to get the wrapped exception for 
+     more details `(ex-cause <error>)`. Defaults to returning logical true when the wrapped exception is either a 
+     java.net.ConnectException or java.net.http.HttpTimeoutException, false otherwise."
   [{:keys [retry-ms-fn on-error-retry-fn? get-url get-opts]
     :or {retry-ms-fn (constantly 5000)
-         on-error-retry-fn? (fn [err] (instance? ConnectException (ex-cause err)))
+         on-error-retry-fn? (fn [err] 
+                              (let [ex (ex-cause err)]
+                                (or (instance? java.net.ConnectException ex)
+                                    (instance? java.net.http.HttpTimeoutException ex))))
          get-opts (constantly nil)}}]
   (let [connections (a/chan)
         close (a/promise-chan)]