-
Notifications
You must be signed in to change notification settings - Fork 54
feat(iota-framework): Add create_auth_info_v1 native function #8477
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
Merged
TheMrAI
merged 10 commits into
vm-lang/aa-auth/8116-feature-branch
from
vm-lang/aa-auth/8117-create_auth_info
Sep 11, 2025
Merged
feat(iota-framework): Add create_auth_info_v1 native function #8477
TheMrAI
merged 10 commits into
vm-lang/aa-auth/8116-feature-branch
from
vm-lang/aa-auth/8117-create_auth_info
Sep 11, 2025
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
A verifier for validating `authenticate` functions introduced for `Move Authentication` (Account Abstraction). Contrary to other verifiers this shouldn't/can't be executed statically. This is why it doesn't appear among the list of other verifiers in: iota_verify_module_metered function. By design it has to be executed during runtime, as it is only executed for a single user specified function. The exact identity of this function will not be known until runtime.
Additionally all relevant components which lacked documentation have been updated.`
|
The latest updates on your projects. Learn more about Vercel for GitHub. 6 Skipped Deployments
|
…l gas Otherwise we would need a new cut, which we don't want to have.
b1ff3fc to
9fb710b
Compare
Thoralf-M
reviewed
Sep 8, 2025
Thoralf-M
reviewed
Sep 8, 2025
iota-execution/latest/iota-verifier/src/account_auth_verifier.rs
Outdated
Show resolved
Hide resolved
Thoralf-M
reviewed
Sep 8, 2025
valeriyr
reviewed
Sep 8, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
The way you updated the documentation comments related to IDs can be used as an example for our future PRs 👍
crates/iota-framework/packages/iota-framework/sources/account.move
Outdated
Show resolved
Hide resolved
iota-execution/latest/iota-verifier/src/account_auth_verifier.rs
Outdated
Show resolved
Hide resolved
theiari
reviewed
Sep 8, 2025
miker83z
approved these changes
Sep 8, 2025
crates/iota-framework/packages/iota-framework/sources/account.move
Outdated
Show resolved
Hide resolved
theiari
reviewed
Sep 8, 2025
iota-execution/latest/iota-verifier/src/account_auth_verifier.rs
Outdated
Show resolved
Hide resolved
valeriyr
approved these changes
Sep 8, 2025
theiari
approved these changes
Sep 8, 2025
9cb883a to
9f9daca
Compare
With this the drop function was removed as well.
9f9daca to
458aa71
Compare
4af2c04
into
vm-lang/aa-auth/8116-feature-branch
40 of 41 checks passed
12 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
sc-platform
Issues related to the Smart Contract Platform group.
vm-language
Issues related to the VM & Language Team
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of change
Introduce create_auth_info_v1 native functions.
This function provides the capability of creating an AuthenticatorInfoV1 pointing to a desired authenticate
function.
The gas costs for the native function haven't been identified yet. Tracking issue: #8433
The native function is not yet tested or integrated into the testing frameworks. Tracking issue: #8416
The possibility of enabling the use of private
authenticatefunction will be investigated here: #8215Links to any relevant issues
fixes #8117
fixes #8216
How the change has been tested
The
create_auth_info_v1feature hasn't been tested yet, but the verifier was in a separate PR.Release Notes