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

Relax semi-honest reveal bounds #796

Merged
merged 1 commit into from
Oct 9, 2023
Merged

Relax semi-honest reveal bounds #796

merged 1 commit into from
Oct 9, 2023

Conversation

akoshelev
Copy link
Collaborator

It shouldn't require fields, semi-honest reveal works with shared values just fine.

noticed in #795

It shouldn't require fields, semi-honest reveal works with shared values just fine.
@akoshelev
Copy link
Collaborator Author

@danielmasny

@akoshelev akoshelev merged commit 689fb51 into main Oct 9, 2023
@akoshelev akoshelev deleted the reveal-shared-value branch October 9, 2023 21:59
@danielmasny
Copy link
Collaborator

we need the malicious reveal for malicious security. The current protocol is targeted towards achieving malicious security in the sense that it only needs a maliciously secure multiplication (and reveal). If we just care about semi honest security, there is a more efficient protocol. There seem to be some changes that we could make to the reveal protocol such that semi honest and malicious has basically the same cost. Then we would not need to distinguish between them and just always use the malicious reveal.

@akoshelev
Copy link
Collaborator Author

I believe our OPRF effort is currently focused around semi-honest security (at least attribution and aggregation steps). It is definitely possible to fix our type system to support both, but for now it could be cleaner if OPRF types that we add did not have to implement traits just for the purpose of conforming to the interface.

It is also possible that the malicious security base types we use will no longer be in place for OPRF approach

@akoshelev akoshelev mentioned this pull request Oct 11, 2023
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.

3 participants