3pp-mirror/penpot
1
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2025-05-29 12:06:12 +02:00
Code Issues Projects Releases Packages Wiki Activity

✨ Add better error report on importing truncated binfile

Browse source
This commit is contained in:
Andrey Antukh 2024-02-13 18:17:43 +01:00
parent afd68fa09d
commit 8ea82021f0
2 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

7
backend/src/app/binfile/v1.clj
View file

@ -40,6 +40,7 @@
[promesa.util :as pu] [promesa.util :as pu]
[yetti.adapter :as yt]) [yetti.adapter :as yt])
(:import (:import
com.github.luben.zstd.ZstdIOException
com.github.luben.zstd.ZstdInputStream com.github.luben.zstd.ZstdInputStream
com.github.luben.zstd.ZstdOutputStream com.github.luben.zstd.ZstdOutputStream
java.io.DataInputStream java.io.DataInputStream
@ -756,6 +757,12 @@
(pu/with-open [input (io/input-stream input)] (pu/with-open [input (io/input-stream input)]
(read-import! (assoc cfg ::input input)))) (read-import! (assoc cfg ::input input))))
(catch ZstdIOException cause
(ex/raise :type :validation
:code :invalid-penpot-file
:hint "invalid penpot file received: probably truncated"
:cause cause))
(catch Throwable cause (catch Throwable cause
(vreset! cs cause) (vreset! cs cause)
(throw cause)) (throw cause))

3
backend/src/app/http/sse.clj
View file

@ -61,9 +61,6 @@
(let [result (handler)] (let [result (handler)]
(events/tap :end result)) (events/tap :end result))
(catch Throwable cause (catch Throwable cause
(binding [l/*context* (errors/request->context request)]
(l/err :hint "unexpected error process streaming response"
:cause cause))
(events/tap :error (errors/handle' cause request))) (events/tap :error (errors/handle' cause request)))
(finally (finally
(sp/close! events/*channel*) (sp/close! events/*channel*)