`Params.InstantiateDefaultPermission` does not support `AccessTypeAnyOfAddresses`
- Dominant language
- Go
- Stars
- 416
- Forks
- 538
- PR merge metrics
- No merged PRs in 30d
Description
The current definition of the `Params` type looks like this
```
message Params {
option (gogoproto.goproto_stringer) = false;
AccessConfig code_upload_access = 1 [
(gogoproto.nullable) = false,
(amino.dont_omitempty) = true,
(gogoproto.moretags) = "yaml:\"code_upload_access\""
];
AccessType instantiate_default_permission = 2
[ (gogoproto.moretags) = "yaml:\"instantiate_default_permission\"" ];
}
```
which is only storing the `AccessType`, which if set to `ACCESS_TYPE_ANY_OF_ADDRESSES`, therefore can't store a list of `addresses`. Just wanted to check this wasn't intentional? Will PR a fix otherwise
Contributor guide
Research direction
Start by locating the Params definition and references to AccessTypeAnyOfAddresses in the wasmd codebase. Determine whether instantiate_default_permission should preserve an address list, then inspect existing tests for permission handling. Done means the setting supports this access type without losing its addresses and the behavior is covered by tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- blockchain
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100