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

make it possible to supply multiple inputs #84

Merged
merged 3 commits into from
Oct 22, 2024
Merged

Conversation

Freax13
Copy link
Owner

@Freax13 Freax13 commented Oct 20, 2024

The key difference between having multiple smaller inputs vs one larger input is that every smaller input is hashed separately as opposed to one hash over the entire input. This makes it possible to easily append to inputs without having to calculate the hash over the entire input. This is very useful in scenarios where a verifier doesn't know an input, but knows its hash. With these changes, it's possible for a verifier to verify an attestation report with multiple such inputs.

There are some use-cases where the output hash is not know ahead of
time, but should be extracted from the attestation report.
verify_and_extract works exactly like verify except that it returns the
output hash instead of getting the output has as a parameter.
@Freax13 Freax13 force-pushed the feature/split-input branch 2 times, most recently from 6ebae27 to 68d38dc Compare October 20, 2024 08:53
The key difference between having multiple smaller inputs vs one larger
input is that every smaller input is hashed separately as opposed to
one hash over the entire input. This makes it possible to easily append
to inputs without having to calculate the hash over the entire input.
This is very useful in scenarios where a verifier doesn't know an
input, but knows its hash. With these changes, it's possible for a
verifier to verify an attestation report with multiple such inputs.
@Freax13 Freax13 force-pushed the feature/split-input branch from 68d38dc to 7f69b11 Compare October 20, 2024 08:59
@Freax13 Freax13 enabled auto-merge October 22, 2024 05:27
@Freax13 Freax13 merged commit f7038d0 into main Oct 22, 2024
50 checks passed
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