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.

Sign up for GitHub

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

Jump to bottom

Add JSON encoding info to hash #19

Closed
taoeffect opened this issue Jul 25, 2023 · 0 comments · Fixed by #21
Closed

Add JSON encoding info to hash #19

taoeffect opened this issue Jul 25, 2023 · 0 comments · Fixed by #21
Assignees
@snowteamer
Labels
help wanted Extra attention is needed priority:high

Comments

@taoeffect
Copy link
Member

Problem

If we ever decide to change the encoding we use we should be able to without things breaking too badly.

Solution

Add that info to the hash, using CIDs if necessary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@snowteamer snowteamer

Labels
help wanted Extra attention is needed priority:high
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use CIDs for content addressing rather than plain hashes snowteamer/chel
2 participants
@taoeffect @snowteamer