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

Entry index parsing improved #44

Merged
merged 1 commit into from
Feb 25, 2025
Merged

Entry index parsing improved #44

merged 1 commit into from
Feb 25, 2025

Conversation

petrpinkas
Copy link
Collaborator

@petrpinkas petrpinkas commented Feb 24, 2025

Entry index is taken from such a text:

Current Root Hash: c41dfd304c856578c70d5e3df30c2ec006b9520d6c318193cd598fcfa14a9188
Entry Hash: 94461160b4b91fc2f18c10ac536a6cd74961d4abf301ec9f8b0889f0a3a276cc
Entry Index in Current Tree: 5
Current Tree Size: 6

Entry index was always set to 0 in the test. This resulted in checking wrong entry when based on entry index.

@petrpinkas petrpinkas requested a review from kdacosta0 February 24, 2025 22:13
@ssulka ssulka merged commit e548425 into main Feb 25, 2025
2 checks passed
@petrpinkas petrpinkas deleted the ppinkas/entry-index branch February 25, 2025 08:57
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.

2 participants