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

Request validation should not throw if verifyingContract is not defined in typed signature #330

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

jpuri
Copy link
Contributor

@jpuri jpuri commented Sep 4, 2024

Creating PR #328 against 14.x branch to expedite release of bug fix.

@jpuri jpuri requested a review from a team as a code owner September 4, 2024 17:32
@jpuri jpuri merged commit 68103ac into 14.x Sep 4, 2024
17 checks passed
@jpuri jpuri deleted the typed_sign_fix branch September 4, 2024 17:34
Gudahtt added a commit that referenced this pull request Sep 4, 2024
* 14.x:
  14.0.1 (#331)
  Request validation should not throw if verifyingContract is not defined in typed signature (#328) (#330)
Gudahtt added a commit that referenced this pull request Sep 6, 2024
* Request validation should not throw if verifyingContract is not defined in typed signature (#328) (#330)

* 14.0.1 (#331)

---------

Co-authored-by: Jyoti Puri <jyotipuri@gmail.com>
Gudahtt added a commit that referenced this pull request Oct 2, 2024
* Request validation should not throw if verifyingContract is not defined in typed signature (#328) (#330)

* 14.0.1 (#331)

* [14.x] fix: support ethermint's EIP712 implementation (#333)

* setting cosmos as allowed string for verifyingContract field

* fixed and linter

* readability

* Update condition to match main branch

* Remove duplicate copy of test

---------

Co-authored-by: Jyoti Puri <jyotipuri@gmail.com>
Co-authored-by: Mark Stacey <markjstacey@gmail.com>

* Version 14.0.2 (#339)

* Version 14.0.2

* Fix typo

Co-authored-by: Michele Esposito <34438276+mikesposito@users.noreply.github.com>

---------

Co-authored-by: Michele Esposito <34438276+mikesposito@users.noreply.github.com>

---------

Co-authored-by: Jyoti Puri <jyotipuri@gmail.com>
Co-authored-by: Michael Tsitrin <114929630+mtsitrin@users.noreply.github.com>
Co-authored-by: Michele Esposito <34438276+mikesposito@users.noreply.github.com>
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.

1 participant