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

LSBBA: ECIES Encryption Version #27

Closed
wants to merge 3 commits into from
Closed

LSBBA: ECIES Encryption Version #27

wants to merge 3 commits into from

Conversation

Oighty
Copy link
Contributor

@Oighty Oighty commented Jan 29, 2024

Alternate encryption scheme that I'm considering to replace the RSA-based one.

The main potential issue I see with RSA is that it may be possible to come up with a combination of (amountOut, seed) that matches an encrypted bid, but that is different than what the user submitted. This should be very difficult, but we have no way to detect it.

In this version, we use a simplified version of the Elliptic Curve Integrated Encryption Scheme (ECIES) where the auction creator provides a public key on the AltBN128 curve. Bidders create a shared key off-chain and conceal it using the public key of the auction. To settle, the private key for the auction can be provided and the encrypted amounts out can be decrypted directly using the AltBN128 ecMul precompile for ~6,000 gas. We use a simple, hash-based key derivation function and XOR encryption, which are weak by themselves, but are likely sufficient behind the EC public key cryptography.

Base automatically changed from batch-auctions to master January 31, 2024 16:29
@Oighty Oighty changed the base branch from master to settle-limit February 15, 2024 15:24
@Oighty
Copy link
Contributor Author

Oighty commented Feb 15, 2024

Superseded by #45

@Oighty Oighty closed this Feb 15, 2024
@0xJem 0xJem deleted the ecies branch March 18, 2024 11:22
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