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

Use Original bytes instead of reserialising #1216

Merged
merged 1 commit into from
Aug 5, 2022
Merged

Conversation

kderme
Copy link
Contributor

@kderme kderme commented Aug 4, 2022

Fixes #1214

We use the SafeToHash class from ledger to extract the initial bytes. Its documentation seems to match what we want

-- | Only Types that preserve their serialisation bytes are members of the
--   class 'SafeToHash'.

@erikd erikd force-pushed the kderme/fix-original-bytes branch from 85b70cf to f5f1be1 Compare August 5, 2022 06:07
@erikd erikd merged commit e0a8470 into master Aug 5, 2022
@iohk-bors iohk-bors bot deleted the kderme/fix-original-bytes branch August 5, 2022 06:58
@kderme kderme mentioned this pull request Sep 11, 2022
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.

Datum round-trip corrupts datum bytes
2 participants