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

[JLS secagg encryption]: encrypt once both model weights and auxiliary variables #1250

Open
ybouilla opened this issue Nov 22, 2024 · 0 comments
Labels
candidate an individual developer submits a work request to the team (extension proposal, bug, other request)

Comments

@ybouilla
Copy link
Contributor

When using secure aggregation, auxiliary variables are encrypted (thanks to #1007 ) along model weights, but the encryption is done separately.
While it should be ok for LOM secagg algorithm, it is more concerning for JLS, where parameters are encrypted using the same key, and could be deciphered through replay attacks.

The goal of this PR is to encrypt both model weights and auxiliary variable at once, and therefore providing a better security when manipulating model and optimizers parameters.

@ybouilla ybouilla added the candidate an individual developer submits a work request to the team (extension proposal, bug, other request) label Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
candidate an individual developer submits a work request to the team (extension proposal, bug, other request)
Projects
None yet
Development

No branches or pull requests

1 participant