Skip to content

Commit

Permalink
Sealevel multisig ISM (#2251)
Browse files Browse the repository at this point in the history
### Description



* Implements the message ID specific version of the multsig ISM (i.e. no
fraud proofs)
* Works by having a single "access control" PDA account (set in the
Initialize fn), which stores the owner
* Setting the set & threshold happens in a single instruction - i.e. no
"add to set" or "remove from set" or "change only threshold", etc
* Tests are split into unit tests & functional tests. Function tests are
for things that involve account creation / CPIs that can't be unit
tested well
* Note that because instructions are usually just borsh serialized
enums, this introduces the expectation that the first instruction of an
ISM program is verify, and the second is the ISM type

### Drive-by changes

* Some small quality of life changes

### Related issues

- Fixes #2216 

### Backward compatibility

_Are these changes backward compatible?_

Yes

_Are there any infrastructure implications, e.g. changes that would
prohibit deploying older commits using this infra tooling?_

None


### Testing

_What kind of testing have these changes undergone?_

Unit Tests & functional tests
  • Loading branch information
tkporter authored May 17, 2023
1 parent acbc347 commit 58063eb
Show file tree
Hide file tree
Showing 31 changed files with 3,175 additions and 258 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
[135,153,145,193,50,88,169,205,206,171,48,1,17,242,3,43,225,72,101,163,93,126,105,165,159,44,243,196,182,240,4,87,22,253,47,198,217,75,23,60,181,129,251,103,140,170,111,35,152,97,16,23,64,17,198,239,79,225,120,141,55,38,60,86]
Loading

0 comments on commit 58063eb

Please sign in to comment.