Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

create_resource: Issue when folder_id is set and not all users keys are imported. #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

danielfl
Copy link

Without importing all users GPG keys, the secret sharing to a folder will fail.

On the share_payload you will receive an error saying the PGP message is empty:
{'user_id': 'XXXXXXXXXXXX', 'data': ''}]}
ERROR:root:{"header":{"id":"XXXX-0bdc-43f0-8d16-XX","status":"error","servertime":1708009984,"action":"XXXX-9d43-5749-a5fe-XX","message":"Could not validate resource data.","url":"/share/resource/XX-7d37-4b13-a86c-XX.json","code":400},"body":{"secrets":{"1":{"data":{"_empty":"The message should not be empty."}}}}

Without importing GPG key, secret sharing to a folder will fail.
@danielfl danielfl changed the title Fix folder sharing issue create_resource: Issue when folder_id is set and not all users keys are imported. Feb 15, 2024
@danielfl
Copy link
Author

danielfl commented Feb 16, 2024

Looking closer, I see this can be achieved by "import_public_keys()", however, I don't want to import/trust hundreds of keys to my keystore, only keys from those I have contact with. But feel free to reject/close this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant