add clj-kondo linting - review

now logging, rather than swallowing, exceptions
cljdoc · Aug 22, 2019 · 9f9bbc2 · 9f9bbc2
1 parent 28c0573
commit 9f9bbc2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/cljdoc/server/api.clj b/src/cljdoc/server/api.clj
@@ -35,7 +35,8 @@
         (ingest/ingest-cljdoc-edn storage data)
         (build-log/api-imported! build-tracker build-id ns-count)
         (build-log/completed! build-tracker build-id))
-      (catch Exception _e
+      (catch Exception e
+        (log/errorf e "analysis job failed for project: %s, version: %s, build-id: %s" project version build-id)
         (build-log/failed! build-tracker build-id "analysis-job-failed")))))
 
 (defn kick-off-build!

diff --git a/src/cljdoc/server/pedestal.clj b/src/cljdoc/server/pedestal.clj
@@ -305,8 +305,8 @@
                                  artifact-id (util/clojars-id params)
                                  group-id group-id)
                    release (try (repos/latest-release-version project)
-                                (catch Exception _e
-                                  (log/warnf "Could not find release for %s" project)))]
+                                (catch Exception e
+                                  (log/warnf e "Could not find release for %s" project)))]
                (->> (if release
                       {:status 302
                        :headers {"Location" (routes/url-for :artifact/version