feat: Re-add password change and reset API endpoints (working version)
Added back password management APIs using handler-case (not with-error-handling): 1. /api/asteroid/user/change-password - Users can change their own password - Requires current password verification - Uses get-current-user() function 2. /api/asteroid/admin/reset-password - Admins can reset any user's password - No current password required - Returns 404 if user not found Both APIs use handler-case for error handling to maintain consistency with other APIs in this file. Build succeeds. Works with password change UI in admin dashboard (admin.ctml, admin.js).
This commit is contained in:
parent
cb13bc9cfd
commit
fd185ed9b1
|
|
@ -105,3 +105,59 @@
|
|||
(api-output `(("status" . "error")
|
||||
("message" . ,(format nil "Error creating user: ~a" e)))
|
||||
:status 500))))
|
||||
|
||||
;; API: Change own password (authenticated users)
|
||||
(define-api asteroid/user/change-password (current-password new-password) ()
|
||||
"API endpoint for users to change their own password"
|
||||
(require-authentication)
|
||||
(handler-case
|
||||
(if (and current-password new-password)
|
||||
(let* ((user (get-current-user))
|
||||
(username (gethash "username" user))
|
||||
(stored-hash (gethash "password-hash" user)))
|
||||
;; Verify current password
|
||||
(if (verify-password current-password
|
||||
(if (listp stored-hash) (first stored-hash) stored-hash))
|
||||
;; Current password is correct, update to new password
|
||||
(if (reset-user-password username new-password)
|
||||
(api-output `(("status" . "success")
|
||||
("message" . "Password changed successfully")))
|
||||
(api-output `(("status" . "error")
|
||||
("message" . "Failed to update password"))
|
||||
:status 500))
|
||||
;; Current password is incorrect
|
||||
(api-output `(("status" . "error")
|
||||
("message" . "Current password is incorrect"))
|
||||
:status 401)))
|
||||
(api-output `(("status" . "error")
|
||||
("message" . "Missing required fields"))
|
||||
:status 400))
|
||||
(error (e)
|
||||
(api-output `(("status" . "error")
|
||||
("message" . ,(format nil "Error changing password: ~a" e)))
|
||||
:status 500))))
|
||||
|
||||
;; API: Reset user password (admin only)
|
||||
(define-api asteroid/admin/reset-password (username new-password) ()
|
||||
"API endpoint for admins to reset any user's password"
|
||||
(require-role :admin)
|
||||
(handler-case
|
||||
(if (and username new-password)
|
||||
(let ((user (find-user-by-username username)))
|
||||
(if user
|
||||
(if (reset-user-password username new-password)
|
||||
(api-output `(("status" . "success")
|
||||
("message" . ,(format nil "Password reset for user: ~a" username))))
|
||||
(api-output `(("status" . "error")
|
||||
("message" . "Failed to reset password"))
|
||||
:status 500))
|
||||
(api-output `(("status" . "error")
|
||||
("message" . ,(format nil "User not found: ~a" username)))
|
||||
:status 404)))
|
||||
(api-output `(("status" . "error")
|
||||
("message" . "Missing required fields"))
|
||||
:status 400))
|
||||
(error (e)
|
||||
(api-output `(("status" . "error")
|
||||
("message" . ,(format nil "Error resetting password: ~a" e)))
|
||||
:status 500))))
|
||||
|
|
|
|||
Loading…
Reference in New Issue