Skip to content

fix: Hash raw key in open_storage_map#1562

Merged
sergerad merged 3 commits intonextfrom
sergerad-hash-raw-key
Jan 21, 2026
Merged

fix: Hash raw key in open_storage_map#1562
sergerad merged 3 commits intonextfrom
sergerad-hash-raw-key

Conversation

@sergerad
Copy link
Collaborator

@sergerad sergerad commented Jan 21, 2026

As per comment:

this method also has a bug because we are not hashing the keys. Let's fix this in a follow-up PR.

@sergerad sergerad added the no changelog This PR does not require an entry in the `CHANGELOG.md` file label Jan 21, 2026
bobbinth
bobbinth approved these changes Jan 21, 2026
View reviewed changes
Copy link
Contributor

@bobbinth bobbinth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thank you!

@sergerad sergerad merged commit 8e32261 into next Jan 21, 2026
7 checks passed
@sergerad sergerad deleted the sergerad-hash-raw-key branch January 21, 2026 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bobbinth bobbinth bobbinth approved these changes

Assignees

No one assigned

Labels

no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sergerad @bobbinth