Penpot/penpot
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2025-01-24 23:49:45 -05:00
Code Issues Activity

🐛 Fix unexpected exception on importing some binary files

Browse source
This commit is contained in:
Andrey Antukh 2023-12-15 17:31:30 +01:00 • committed by Andrés Moya
parent 60fc1a48a5
commit 15f81e557c
3 changed files with 9 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
backend/src/app/http/errors.clj
View file

@ -81,6 +81,7 @@
(cond
(or (= code :spec-validation)
(= code :params-validation)
(= code :schema-validation)
(= code :data-validation))
(let [explain (ex/explain data)]
{::rres/status 400

3
backend/src/app/rpc/commands/binfile.clj
View file

@ -722,7 +722,8 @@
(-> fdata
(update :pages-index relink-shapes)
(update :components relink-shapes)
(update :media relink-media))))))
(update :media relink-media)
(d/without-nils))))))
(defmethod read-section :v1/files

15
frontend/src/app/worker/import.cljs
View file

@ -726,16 +726,13 @@
(log/error :hint "unexpected error on import process"
:project-id project-id
::log/sync? true)
(when (map? cause)
(let [edata (if (map? cause) cause (ex-data cause))]
(println "Error data:")
(pp/pprint (dissoc cause :explain) {:level 2 :length 10}))
(pp/pprint (dissoc edata :explain) {:level 2 :length 10})
(when (string? (:explain cause))
(js/console.log (:explain cause)))
(when (string? (:explain edata))
(js/console.log (:explain edata)))
(rx/of {:status :import-error
:file-id (:file-id data)
:error (:hint cause)
:error-data cause}))))))))))
(rx/of {:status :import-error
:file-id (:file-id data)})))))))))))