TrueCloudLab/frostfs-node
19
1
Fork 27
Code Issues 91 Pull requests 8 Projects Releases 1 Activity Actions

Fix delete container signature check #745

Merged
dstepanov-yadro merged 1 commit from dstepanov-yadro/frostfs-node:fix/remove_container_debugee into master 2023-10-19 15:12:49 +00:00
Conversation 0 Commits 1 Files changed 1 +1 -1
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit dc4d27201b - Show all commits

2
pkg/morph/client/container/delete.go
View file

@ -67,7 +67,7 @@ func (d *DeletePrm) SetKey(key []byte) {
// //
// If TryNotary is provided, calls notary contract. // If TryNotary is provided, calls notary contract.
func (c *Client) Delete(p DeletePrm) error { func (c *Client) Delete(p DeletePrm) error {
if len(p.signature) == 0 && p.IsControl() { if len(p.signature) == 0 && !p.IsControl() {
dstepanov-yadro commented 2023-10-19 15:04:46 +00:00
Outdated
Review
Copy link

When IsControl == true, then len(p.signature) == 0, because there is no container owner signature.

When IsControl == true, then len(p.signature) == 0, because there is no container owner signature.
return errNilArgument return errNilArgument
} }