Add public key as argument for container_contract.Delete() method #28
No reviewers
TrueCloudLab/storage-core-committers
TrueCloudLab/storage-services-developers
Labels
No Label
P0
P1
P2
P3
good first issue
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No Milestone
No Assignees
5 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-contract#28
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "acid-ant/frostfs-contract:feature/27-cnt-del-add-arg"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
Close #27
Signed-off-by: Anton Nikiforov an.nikiforov@yadro.com
@ -311,3 +311,3 @@
//
// If the container doesn't exist, it panics with NotFoundError.
func Delete(containerID []byte, signature interop.Signature, token []byte) {
func Delete(containerID []byte, signature interop.Signature, token []byte, publicKey interop.PublicKey) {
What about making it the third parameter (like in
Put
)?Done.
dc0e6c5e73
to2da0ff5750
@ -311,3 +311,3 @@
//
// If the container doesn't exist, it panics with NotFoundError.
func Delete(containerID []byte, signature interop.Signature, token []byte) {
func Delete(containerID []byte, signature interop.Signature, publicKey interop.PublicKey, token []byte) {
It should break compatibility with current version of the node. For the Neo VM
Delete()
with 3 arguments andDelete()
with 4 arguments are completely different methods.I suggest to add one more method and reuse the code of
Delete()
method. SeePut()
andPutNamed()
methods. So newer versions of node will use new method and older versions stick with currentDelete()
method.However, if we don't care about compatibility there for some reason, this is okay.
We can mark it as deprecated and use https://github.com/nspcc-dev/neo-go/blob/master/docs/compiler.md#Overloads .
We can mark it as deprecated and use https://github.com/nspcc-dev/neo-go/blob/master/docs/compiler.md#Overloads .
Let's break compatibility because we don't have any public networks to be compatible with.