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

feat: add ed25519_verify host function #1010

Merged
merged 14 commits into from
Jun 14, 2023

Conversation

medicz
Copy link
Contributor

@medicz medicz commented Mar 19, 2023

No description provided.

Copy link
Member

@ailisp ailisp left a comment

Choose a reason for hiding this comment

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

Thank you @medicz. This enhancement looks good to me. Could you fix the CI?

@medicz
Copy link
Contributor Author

medicz commented Mar 28, 2023

Thank you @medicz. This enhancement looks good to me. Could you fix the CI?

lint CI fixed, audit CI I probably need help with, since it fails on dependencies coming from near-vm-logic, any ideas there?

@medicz medicz force-pushed the ed25519_verify branch 2 times, most recently from 679284b to 05d5b42 Compare March 28, 2023 15:11
@chefsale
Copy link
Contributor

cc: @miraclx @ailisp @ChaoticTempest Can you please help with the above blocker?

@miraclx
Copy link
Contributor

miraclx commented Mar 29, 2023

Working with SRE to have the rust toolchain inside the buildkite images updated. Will retry the CI run once updated.

@medicz
Copy link
Contributor Author

medicz commented Apr 13, 2023

@miraclx audit CI fails due to dependency inherited from near-vm-logic 0.16.1. Any advice about this?

@miraclx
Copy link
Contributor

miraclx commented Apr 13, 2023

I'll be taking a look at this today! Fingers crossed, this can just be a cargo update resolution and not a full version bump, but I'll report back.

@chefsale
Copy link
Contributor

Seems like checks are passing now. @miraclx @medicz @ailisp @itegulov @ChaoticTempest Can we land this and publish it?

@medicz
Copy link
Contributor Author

medicz commented Apr 25, 2023

@miraclx any updates about audit CI?

@miraclx
Copy link
Contributor

miraclx commented May 4, 2023

Apologies for the delay. The Consensus crunch had us prioritize other tasks. But I'm back on this. The status is I'm updating nearcore dependencies to alleviate the vulnerabilities downstream, afterwards, I'll cut a new release of nearcore crates, and revert back. But for clarity, this is being worked on.

@miraclx
Copy link
Contributor

miraclx commented May 15, 2023

As far as updates go;

@miraclx
Copy link
Contributor

miraclx commented May 30, 2023

26 days, 7 PRs (to nearcore), 2,056 additions and 667 deletions (to my fork of cargo-workspaces) later, we finally have nearcore's automated publishing process back in the green.

And have just published v0.17.0 of all nearcore primitives.

https://github.com/near/nearcore/releases/tag/crates-0.17.0

@medicz
Copy link
Contributor Author

medicz commented May 31, 2023

@miraclx with introduction of primitives v0.17.0, buildkite needs to be updated to use rust 1.69

@miraclx
Copy link
Contributor

miraclx commented May 31, 2023

As much as I would've loved to see this through, my time with Pagoda has come to an end.

Hey @frol, can you reach out to SRE to handle this?

@frol
Copy link
Collaborator

frol commented May 31, 2023

@medicz I reached out to the SRE team to update Rust on the CI. Yet, there are other issues with this PR that don't seem to be related to the Rust version.

@medicz
Copy link
Contributor Author

medicz commented May 31, 2023

@medicz I reached out to the SRE team to update Rust on the CI. Yet, there are other issues with this PR that don't seem to be related to the Rust version.

indeed, some of those were related to old Cargo.lock being present. Updated in last commit.

near-sdk/src/test_utils/context.rs Outdated Show resolved Hide resolved
near-sdk/src/test_utils/context.rs Outdated Show resolved Hide resolved
near-sdk/src/environment/mock/mocked_blockchain.rs Outdated Show resolved Hide resolved
near-sdk/src/environment/mock/mocked_blockchain.rs Outdated Show resolved Hide resolved
near-sdk/src/environment/mock/mocked_blockchain.rs Outdated Show resolved Hide resolved
@frol
Copy link
Collaborator

frol commented Jun 6, 2023

I finally found a way to update Rust on buildkite (https://buildkite.com/nearprotocol/near-sdk-rs/steps), so it is running now. Still, there are some other changes that I requested above that are important to maintain backward compatibility.

@medicz
Copy link
Contributor Author

medicz commented Jun 6, 2023

I finally found a way to update Rust on buildkite (https://buildkite.com/nearprotocol/near-sdk-rs/steps), so it is running now. Still, there are some other changes that I requested above that are important to maintain backward compatibility.

@frol PR was updated

@frol
Copy link
Collaborator

frol commented Jun 7, 2023

@medicz Please, fix CI failures

@medicz
Copy link
Contributor Author

medicz commented Jun 13, 2023

@medicz Please, fix CI failures

fix will have to wait, since problem is deserialisation of contracts on rust 1.70 (see: near/nearcore#9140 (comment) and https://near.org/near/widget/PostPage?accountId=pagodaplatform.near&blockHeight=93538986).

I have also verified that same issue is currently happening on master branch as well.

@frol
Copy link
Collaborator

frol commented Jun 13, 2023

@medicz Let's pin Rust version to 1.69.0 in the GitHub Actions

@medicz
Copy link
Contributor Author

medicz commented Jun 13, 2023

@medicz Let's pin Rust version to 1.69.0 in the GitHub Actions

@frol pinned

near-sdk/src/environment/mock/mocked_blockchain.rs Outdated Show resolved Hide resolved
Co-authored-by: Vlad Frolov <frolvlad@gmail.com>
@frol frol merged commit 42632a0 into near:master Jun 14, 2023
@medicz
Copy link
Contributor Author

medicz commented Jun 14, 2023

@frol can 4.1.2 be released? It is a blocker for us.

@frol
Copy link
Collaborator

frol commented Jun 15, 2023

@medicz I am working on it. Naive release suggests that there are breaking changes and semver suggests we release 5.0, which I would prefer to avoid doing right now, so I am trying to minimize the scope and see if we can still cut 4.x release.

@ijerkovic
Copy link

@frol any news on when the new near-sdk will be released?

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.

6 participants