CosmWasm / CosmWasm/wasmd

`Params.InstantiateDefaultPermission` does not support `AccessTypeAnyOfAddresses`

Open
#1,501 6 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.