johanels/penpot
1
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2025-12-11 22:14:05 +01:00
Code Issues Actions 9 Packages Projects Releases Wiki Activity

✨ Remove unnecesary report on duplicate email error validation

Browse Source
This commit is contained in:
Andrey Antukh
2025-11-04 20:33:58 +01:00
parent c850f101d3
commit 9b3f68ad14
2 changed files with 12 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
backend/src/app/db.clj
View File

@@ -704,6 +704,12 @@
(and (sql-exception? cause) (and (sql-exception? cause)
(= "40001" (.getSQLState ^java.sql.SQLException cause)))) (= "40001" (.getSQLState ^java.sql.SQLException cause))))
(defn duplicate-key-error?
[cause]
(and (sql-exception? cause)
(= "23505" (.getSQLState ^java.sql.SQLException cause))))
(extend-protocol jdbc.prepare/SettableParameter (extend-protocol jdbc.prepare/SettableParameter
clojure.lang.Keyword clojure.lang.Keyword
(set-parameter [^clojure.lang.Keyword v ^PreparedStatement s ^long i] (set-parameter [^clojure.lang.Keyword v ^PreparedStatement s ^long i]

15
backend/src/app/rpc/commands/auth.clj
View File

@@ -315,16 +315,13 @@
(-> (db/insert! conn :profile params) (-> (db/insert! conn :profile params)
(profile/decode-row)) (profile/decode-row))
(catch org.postgresql.util.PSQLException cause (catch org.postgresql.util.PSQLException cause
(let [state (.getSQLState cause)] (if (db/duplicate-key-error? cause)
(if (not= state "23505") (ex/raise :type :validation
(throw cause) :code :email-already-exists
:hint "email already exists"
:cause cause)
(throw cause))))))
(do
(l/error :hint "not an error" :cause cause)
(ex/raise :type :validation
:code :email-already-exists
:hint "email already exists"
:cause cause))))))))
(defn create-profile-rels! (defn create-profile-rels!
[conn {:keys [id] :as profile}] [conn {:keys [id] :as profile}]