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

Ergonomic MST #4930

Closed
mversic opened this issue Aug 2, 2024 · 7 comments · Fixed by #5027
Closed

Ergonomic MST #4930

mversic opened this issue Aug 2, 2024 · 7 comments · Fixed by #5027
Assignees

Comments

@mversic
Copy link
Contributor

mversic commented Aug 2, 2024

Ideas on how can we make MST more ergonomic:

  1. All smart contracts for MST should be registered from Executor, i.e. be put in default executor migration
  2. I think we need an API to fetch all currently voted for transactions, i.e. pending transactions
  3. we need a way to expire MST that have not been active for a while
  4. We can define custom instructions for dealing with MST and also have them defined as part of default executor

Additionally, we should consider adding (behind a feature flag "mst") a specialized API to both iroha client and cli for dealing with MSTs.

@mversic
Copy link
Contributor Author

mversic commented Aug 5, 2024

actually, instead of doing it during executor migration as was suggested in 1. they should be registered in the genesis

@s8sato s8sato self-assigned this Aug 20, 2024
@nxsaken nxsaken added this to Iroha Sep 4, 2024
@nxsaken nxsaken moved this to Backlog in Iroha Sep 4, 2024
@nxsaken nxsaken moved this from Backlog to Work in Progress in Iroha Sep 4, 2024
@mversic
Copy link
Contributor Author

mversic commented Oct 18, 2024

* [make the default multisig ttl indefinite?](https://github.com/hyperledger/iroha/pull/5027#discussion_r1742874965)

no, there should always be some default non-infinite value

@0x009922
Copy link
Contributor

My comment regarding the client interface: #5027 (comment)

I don't immediately like the idea of having this being included directly into the client. Though, it depends on how much we need to make multisig an official out-of-the-box feature.

As an alternative idea, there could a separate CLI to construct multisig-related instructions in a form of JSON, which then could be supplied to the client:

iroha_multisig register --account .. --weights .. --quorum .. | iroha json transaction

@s8sato
Copy link
Contributor

s8sato commented Oct 29, 2024

Accidentally automatically closed. Reopened

@s8sato
Copy link
Contributor

s8sato commented Nov 6, 2024

The following will be addressed after #5217:

  • test: recursion test suites
  • refactor!: consolidate metadata read/write
  • fix: prevent expired/executed orphans
  • fix: question existence and format of metadata
  • feat!: override TTL per transaction
  • feat: pretty list of pending proposals

@s8sato
Copy link
Contributor

s8sato commented Nov 28, 2024

Closed as completed by:

Once settled the multisig issue of where to bridge the gap between the internal representation and the ergonomic interface. API may change depending on future requirements

@s8sato s8sato closed this as completed Nov 28, 2024
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 a pull request may close this issue.

3 participants