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

15.1.0 #352

Merged
merged 3 commits into from
Dec 18, 2024
Merged

15.1.0 #352

merged 3 commits into from
Dec 18, 2024

Conversation

github-actions[bot]
Copy link
Contributor

This is the release candidate for version 15.1.0.

github-actions and others added 2 commits December 18, 2024 15:17
@jpuri jpuri marked this pull request as ready for review December 18, 2024 15:18
@jpuri jpuri requested a review from a team as a code owner December 18, 2024 15:18
CHANGELOG.md Outdated
@@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

## [15.1.0]
### Changed
- fix: Add validation for types sign message primary type ([#350](https://github.com/MetaMask/eth-json-rpc-middleware/pull/350))
Copy link
Member

Choose a reason for hiding this comment

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

This is a breaking change, right? We're throwing errors in this middleware in cases we weren't previously. There was no way the signature would actually work in those cases though, so maybe this is fine to categorize as non-breaking 🤔

Copy link
Member

@Gudahtt Gudahtt Dec 18, 2024

Choose a reason for hiding this comment

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

Could you improve the change entry? The fix: prefix is confusing given that this is under the "Changed" category, and is a minor bump (which suggests that this is not just a bug fix).

Maybe we can describe this as "Improved validation of primary type for signTypedDataV3 and signTypedDataV4"

Copy link
Contributor

Choose a reason for hiding this comment

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

Sounds good, I updated the PR.

@jpuri jpuri requested review from Gudahtt and jpuri and removed request for jpuri December 18, 2024 17:18
Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@jpuri jpuri merged commit 9db63df into main Dec 18, 2024
19 checks passed
@jpuri jpuri deleted the release/15.1.0 branch December 18, 2024 22:39
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