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

feat: add APIs for third-party certifications #349

Merged
merged 1 commit into from May 13, 2024

Conversation

hko-s
Copy link
Contributor

@hko-s hko-s commented May 10, 2024

Fixes #347

cc: @kyrias

@dignifiedquire
Copy link
Member

Api looks good to me, needs a conflict resolution and ideally also a test that signs and verifies such a third party construction

@hko-s
Copy link
Contributor Author

hko-s commented May 11, 2024

Api looks good to me, needs a conflict resolution and ideally also a test that signs and verifies such a third party construction

Cool, thanks! I'll add a test soon.

@hko-s hko-s force-pushed the third-party-sig branch 2 times, most recently from fbdbb49 to b71bc3e Compare May 12, 2024 21:59
@hko-s
Copy link
Contributor Author

hko-s commented May 12, 2024

Api looks good to me, needs a conflict resolution and ideally also a test that signs and verifies such a third party construction

Added a test for UserId (and APIs for verification of third-party certifications).
But I can't decide if I find the added noise for a UserAttribute unit test worth it %)

@dignifiedquire dignifiedquire merged commit ab39417 into rpgp:master May 13, 2024
18 checks passed
@hko-s hko-s deleted the third-party-sig branch May 13, 2024 19:55
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.

Should add an API to issue third-party certifications (e.g. over a UserId)
2 participants