DELETE /api/ciphers not implemented #2363

Closed
opened 2025-10-09 18:01:35 +03:00 by OVERLORD · 7 comments
Owner

Originally created by @mprasil on GitHub.

New vault (#93) calls DELETE on /api/ciphers to "Delete selected" ciphers. We have POST handler already implemented for this, so this should be easy to fix.

Originally created by @mprasil on GitHub. New vault (#93) calls `DELETE` on `/api/ciphers` to "Delete selected" ciphers. We have `POST` handler [already implemented](https://github.com/dani-garcia/bitwarden_rs/blob/48e69cebab29585684be4bf22282ff1dd4697d24/src/api/core/ciphers.rs#L465) for this, so this should be easy to fix.
OVERLORD added the future Vaultgood first issuebug labels 2025-10-09 18:01:35 +03:00
Author
Owner

@mannp commented on GitHub:

Is the stable release of v2 tagged for the docker image at all?

Currently using the beta build to get v2, but unclear if there is another build I should be using?

Thanks

@mannp commented on GitHub: Is the stable release of v2 tagged for the docker image at all? Currently using the beta build to get v2, but unclear if there is another build I should be using? Thanks
Author
Owner

@mannp commented on GitHub:

Ok, so i updated my beta to the latest and it still doesn't delete any entries.

@mannp commented on GitHub: Ok, so i updated my beta to the latest and it still doesn't delete any entries.
Author
Owner

@mprasil commented on GitHub:

This is why this issue was open. 😄 It hasn't been fixed yet, I think I've confused you somewhere into thinking it's been resolved.

@mprasil commented on GitHub: This is why this issue was open. 😄 It hasn't been fixed yet, I think I've confused you somewhere into thinking it's been resolved.
Author
Owner

@mprasil commented on GitHub:

The beta image is using the 2.0.0 release of Vault, so that's the one you can use to test it.

@mprasil commented on GitHub: The beta image is using the 2.0.0 release of Vault, so that's the one you can use to test it.
Author
Owner

@mannp commented on GitHub:

Apologies for the noise, thought I'd seen a post saying 3 or 4 bugs were fixed, including this one and there were in a stable branch of v2.

@mannp commented on GitHub: Apologies for the noise, thought I'd seen a post saying 3 or 4 bugs were fixed, including this one and there were in a stable branch of v2.
Author
Owner

@mprasil commented on GitHub:

Yeah, you'll see these issues being closed once that is fixed.

@mprasil commented on GitHub: Yeah, you'll see these issues being closed once that is fixed.
Author
Owner

@mprasil commented on GitHub:

Resolved in #115

@mprasil commented on GitHub: Resolved in #115
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/vaultwarden#2363