acid-ant/frostfs-contract
1
0
Fork 0
forked from TrueCloudLab/frostfs-contract
Code Issues Pull requests Projects Releases Packages Wiki Activity

[#121] container: Reword panic messages for better error wrapping

Browse source 
Signed-off-by: Alex Vanin <alexey@nspcc.ru>
This commit is contained in:
Alex Vanin 2021-09-20 17:23:19 +03:00 committed by Alex Vanin
parent ea89844033
commit b45a4162b4
1 changed files with 11 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
container/container_contract.go
View file

@ -80,7 +80,7 @@ func _deploy(data interface{}, isUpdate bool) {
} }
if len(addrNetmap) != 20 || len(addrBalance) != 20 || len(addrID) != 20 { if len(addrNetmap) != 20 || len(addrBalance) != 20 || len(addrID) != 20 {
panic("init: incorrect length of contract script hash") panic("incorrect length of contract script hash")
} }
storage.Put(ctx, common.OwnerKey, owner) storage.Put(ctx, common.OwnerKey, owner)
@ -305,7 +305,7 @@ func SetEACL(eACL []byte, signature interop.Signature, publicKey interop.PublicK
ownerID := getOwnerByID(ctx, containerID) ownerID := getOwnerByID(ctx, containerID)
if len(ownerID) == 0 { if len(ownerID) == 0 {
panic("setEACL: container does not exists") panic("container does not exist")
} }
var ( // for invocation collection without notary var ( // for invocation collection without notary
@ -362,7 +362,7 @@ func EACL(containerID []byte) ExtendedACL {
ownerID := getOwnerByID(ctx, containerID) ownerID := getOwnerByID(ctx, containerID)
if len(ownerID) == 0 { if len(ownerID) == 0 {
panic("eACL: container does not exists") panic("container does not exist")
} }
return getEACL(ctx, containerID) return getEACL(ctx, containerID)
@ -375,11 +375,11 @@ func PutContainerSize(epoch int, cid []byte, usedSize int, pubKey interop.Public
ctx := storage.GetContext() ctx := storage.GetContext()
if !runtime.CheckWitness(pubKey) { if !runtime.CheckWitness(pubKey) {
panic("container: invalid witness for size estimation") panic("invalid witness of container size estimation")
} }
if !isStorageNode(ctx, pubKey) { if !isStorageNode(ctx, pubKey) {
panic("container: only storage nodes can save size estimations") panic("putContainerSize method must be invoked by storage node from network map")
} }
key := estimationKey(epoch, cid, pubKey) key := estimationKey(epoch, cid, pubKey)
@ -457,12 +457,12 @@ func NewEpoch(epochNum int) {
netmapContractKey, netmapContractKey,
) )
if !indirectCall { if !indirectCall {
panic("newEpoch: this method must be invoked from inner ring") panic("newEpoch method must be invoked by inner ring")
} }
} else { } else {
multiaddr := common.AlphabetAddress() multiaddr := common.AlphabetAddress()
if !runtime.CheckWitness(multiaddr) { if !runtime.CheckWitness(multiaddr) {
panic("newEpoch: this method must be invoked from inner ring") panic("newEpoch method must be invoked by inner ring")
} }
} }
@ -487,12 +487,12 @@ func StartContainerEstimation(epoch int) {
alphabet = common.AlphabetNodes() alphabet = common.AlphabetNodes()
nodeKey = common.InnerRingInvoker(alphabet) nodeKey = common.InnerRingInvoker(alphabet)
if len(nodeKey) == 0 { if len(nodeKey) == 0 {
panic("startEstimation: only inner ring nodes can invoke this") panic("startContainerEstimation method must be invoked by inner ring")
} }
} else { } else {
multiaddr := common.AlphabetAddress() multiaddr := common.AlphabetAddress()
if !runtime.CheckWitness(multiaddr) { if !runtime.CheckWitness(multiaddr) {
panic("startEstimation: only inner ring nodes can invoke this") panic("startContainerEstimation method must be invoked by inner ring")
} }
} }
@ -527,12 +527,12 @@ func StopContainerEstimation(epoch int) {
alphabet = common.AlphabetNodes() alphabet = common.AlphabetNodes()
nodeKey = common.InnerRingInvoker(alphabet) nodeKey = common.InnerRingInvoker(alphabet)
if len(nodeKey) == 0 { if len(nodeKey) == 0 {
panic("stopEstimation: only inner ring nodes can invoke this") panic("stopContainerEstimation method must be invoked by inner ring")
} }
} else { } else {
multiaddr := common.AlphabetAddress() multiaddr := common.AlphabetAddress()
if !runtime.CheckWitness(multiaddr) { if !runtime.CheckWitness(multiaddr) {
panic("stopEstimation: only inner ring nodes can invoke this") panic("stopContainerEstimation method must be invoked by inner ring")
} }
} }