CosmWasm / CosmWasm/wasmd

Update Instantiate Config logic

Aperta
#1,663 2 commenti 1 reazione 0 assegnatari Vedi su GitHub
discussion
Lingua principale
Go
Stelle
416
Fork
538
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Hi,

I've been trying to update the instantiate config to move from a multisig being allowed to a multisig + DAO, to me seems counterintuitive that this is not possible when the multisig is the "creator"/admin of the code.

```go
"anyOf < anyOf": {
superSet: AccessConfig{Permission: AccessTypeAnyOfAddresses, Addresses: []string{"owner"}},
check: AccessConfig{Permission: AccessTypeAnyOfAddresses, Addresses: []string{"other"}},
isSubSet: true,
},
"anyOf(multiple) < anyOf": {
superSet: AccessConfig{Permission: AccessTypeAnyOfAddresses, Addresses: []string{"owner"}},
check: AccessConfig{Permission: AccessTypeAnyOfAddresses, Addresses: []string{"owner", "other"}},
isSubSet: true,
},
```
I added a simple test and this 2 will fail, and I think is the IsSubSet logic is applying the checks correctly but not sure if this logic should be applied when the owner is the `creator` of the code at least not when trying to update the config.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Leggi prima la logica di IsSubSet e i controlli di accesso di instantiate-config, quindi riproduci i due casi anyOf dell’issue nel semplice test menzionato. La semantica prevista di creator/admin deve essere confermata prima dell’implementazione; il lavoro è completato quando il comportamento concordato della configurazione multisig e DAO è coperto dai test.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
go
Ambito
blockchain
Tipo di issue
Funzionalità
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
28/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.