[GH-ISSUE #100] PUT /api/ciphers/share not implemented #57

Closed
opened 2026-03-03 01:24:18 +03:00 by kerem · 10 comments
Owner

Originally created by @mprasil on GitHub (Jul 21, 2018).
Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/100

New vault (#93) calls PUT on /api/ciphers/share to "Share selected" ciphers. This functionality wasn't present in the previous vault so implementation will be a bit more involved, but should be pretty similar to "move selected".

(also woohoo! 100th issue 😄)

Originally created by @mprasil on GitHub (Jul 21, 2018). Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/100 New vault (#93) calls `PUT` on `/api/ciphers/share` to "Share selected" ciphers. This functionality wasn't present in the previous vault so implementation will be a bit more involved, but should be pretty similar to "move selected". (also woohoo! 100th issue :smile:)
Author
Owner

@tgturner commented on GitHub (Jul 26, 2018):

I would love to take a look at this one!

<!-- gh-comment-id:408070081 --> @tgturner commented on GitHub (Jul 26, 2018): I would love to take a look at this one!
Author
Owner

@mprasil commented on GitHub (Jul 26, 2018):

Go for it and don't hesitate to ask for help.

<!-- gh-comment-id:408082367 --> @mprasil commented on GitHub (Jul 26, 2018): Go for it and don't hesitate to ask for help.
Author
Owner

@tgturner commented on GitHub (Jul 26, 2018):

Thanks! I'll take a look at it this evening.

<!-- gh-comment-id:408102871 --> @tgturner commented on GitHub (Jul 26, 2018): Thanks! I'll take a look at it this evening.
Author
Owner

@krankur commented on GitHub (Aug 1, 2018):

I guess we need to add an organization in order to test the "Share" functionality, but I am not able to figure out how to add an organization? I tried uploading a random JSON file as the license file but I am getting a 401 error. Can someone please guide me on this?

<!-- gh-comment-id:409528307 --> @krankur commented on GitHub (Aug 1, 2018): I guess we need to add an organization in order to test the "Share" functionality, but I am not able to figure out how to add an organization? I tried uploading a random JSON file as the license file but I am getting a 401 error. Can someone please guide me on this?
Author
Owner

@mprasil commented on GitHub (Aug 1, 2018):

Hi, there's no licence required - just create free organization (the first option) and it should have all features after that.

<!-- gh-comment-id:409529384 --> @mprasil commented on GitHub (Aug 1, 2018): Hi, there's no licence required - just create free organization (the first option) and it should have all features after that.
Author
Owner

@dani-garcia commented on GitHub (Aug 1, 2018):

I think that happens in the self hosted version (when you build with npm run dist:selfhost)

It should work if you build the normal version (with npm run dist)

<!-- gh-comment-id:409529396 --> @dani-garcia commented on GitHub (Aug 1, 2018): I think that happens in the self hosted version (when you build with npm run dist:selfhost) It should work if you build the normal version (with npm run dist)
Author
Owner

@mprasil commented on GitHub (Aug 1, 2018):

@dani-garcia I submitted #109 to merge recent changes from master. We should do this before @krankur touches the sharing API as there were some major changes and will be pain to merge later.

<!-- gh-comment-id:409532194 --> @mprasil commented on GitHub (Aug 1, 2018): @dani-garcia I submitted #109 to merge recent changes from master. We should do this before @krankur touches the sharing API as there were some major changes and will be pain to merge later.
Author
Owner

@mprasil commented on GitHub (Aug 1, 2018):

Thanks for merging it. @krankur make sure you're working with latest beta code.

<!-- gh-comment-id:409536730 --> @mprasil commented on GitHub (Aug 1, 2018): Thanks for merging it. @krankur make sure you're working with latest beta code.
Author
Owner

@krankur commented on GitHub (Aug 1, 2018):

@mprasil Will do. Thanks for the heads up.

<!-- gh-comment-id:409559173 --> @krankur commented on GitHub (Aug 1, 2018): @mprasil Will do. Thanks for the heads up.
Author
Owner

@mprasil commented on GitHub (Aug 9, 2018):

I think this was resolved in #116

<!-- gh-comment-id:411814957 --> @mprasil commented on GitHub (Aug 9, 2018): I think this was resolved in #116
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/vaultwarden#57
No description provided.