Skip to content

Commit 7d90fb8

Browse files
committed
fix(passkeys): return 204 when deleting a passkey
1 parent f109550 commit 7d90fb8

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

internal/api/passkey_manage.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,8 @@ func (a *API) PasskeyDelete(w http.ResponseWriter, r *http.Request) error {
144144
return apierrors.NewInternalServerError("Database error deleting passkey").WithInternalError(err)
145145
}
146146

147-
return sendJSON(w, http.StatusOK, toPasskeyListItem(cred))
147+
w.WriteHeader(http.StatusNoContent)
148+
return nil
148149
}
149150

150151
func toPasskeyListItem(cred *models.WebAuthnCredential) PasskeyListItem {

internal/api/passkey_manage_test.go

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -201,11 +201,8 @@ func (ts *PasskeyTestSuite) TestPasskeyDelete() {
201201
token := ts.generateToken(ts.TestUser, &ts.TestSession.ID)
202202
w := ts.makeRequest(http.MethodDelete, fmt.Sprintf("http://localhost/passkeys/%s", cred.ID), nil, withBearerToken(token))
203203

204-
ts.Equal(http.StatusOK, w.Code)
205-
206-
var item PasskeyListItem
207-
require.NoError(ts.T(), json.NewDecoder(w.Body).Decode(&item))
208-
ts.Equal(cred.ID.String(), item.ID)
204+
ts.Equal(http.StatusNoContent, w.Code)
205+
ts.Empty(w.Body.Bytes())
209206

210207
// Verify deleted from database
211208
_, err := models.FindWebAuthnCredentialByID(ts.API.db, cred.ID)

0 commit comments

Comments
 (0)