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

fix: Remove account_id validation from new_unvalidated() #20

Merged
merged 2 commits into from
Nov 24, 2023

Conversation

nikurt
Copy link
Contributor

@nikurt nikurt commented Nov 24, 2023

That function even explicitly says that it does no validation and that the validation is responsibility of the caller.

The validation is a problem for executing testnet receipt 9dTn32kX1gKFoP5YaeQKRs98QrcArbpYfB17ou3CNdML in chunk 5BNxaoUPWN7FaPNcUUPS17XXkM6zNh4bKmLAGNhhunN5 at block height 147314170

That function even explicitly says that it does no validation and that the validation is responsibility of the caller.
@nikurt nikurt changed the title Remove account_id validation from new_unvalidated() fix: Remove account_id validation from new_unvalidated() Nov 24, 2023
@nikurt nikurt requested a review from frol November 24, 2023 13:45
Copy link
Collaborator

@frol frol left a comment

Choose a reason for hiding this comment

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

Let's only disable it for nearcore

@frol frol merged commit 26cc387 into main Nov 24, 2023
5 checks passed
@frol frol deleted the nikurt-new-unvalidated branch November 24, 2023 16:22
frol added a commit that referenced this pull request Nov 24, 2023
## 🤖 New release
* `near-account-id`: 1.0.0-alpha.3 -> 1.0.0-alpha.4 (✓ API compatible
changes)

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[1.0.0-alpha.4](v1.0.0-alpha.3...v1.0.0-alpha.4)
- 2023-11-24

### Fixed
- Remove account_id validation from `new_unvalidated()` when
`internal_unstable` feature is enabled (required by nearcore)
([#20](#20))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/MarcoIeni/release-plz/).

Signed-off-by: Vlad Frolov <304265+frol@users.noreply.github.com>
Co-authored-by: Vlad Frolov <304265+frol@users.noreply.github.com>
github-merge-queue bot pushed a commit to near/nearcore that referenced this pull request Nov 24, 2023
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