mirror of
https://github.com/nspcc-dev/neofs-contract.git
synced 2026-03-01 04:28:59 +00:00
What should happen when domain expires #97
Labels
No labels
I1
I2
I3
I4
S1
S2
S3
S4
U1
U2
U3
U4
alphabet
audit
balance
blocked
bug
config
container
discussion
documentation
enhancement
feature
go
good first issue
help wanted
neofs
neofsid
netmap
nns
nns
performance
proxy
question
reputation
security
task
test
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
nspcc-dev/neofs-contract#97
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @alexvanin on GitHub (Sep 16, 2022).
I suggest to discuss it there and then move issue to a proper repository.
All domains, including top level domains, have expiration parameter. One day sooner or later all domains will expire. Now NNS stores in domain records:
According to the code, container alias should live for 1 week. Not sure it works this way, but anyway.
nspcc-dev/neofs-contract@70ae2c94e1/container/container_contract.go (L79)I can't find any
Renewinvocation by alphabet nodes. I suppose alphabet nodes should create multi signed transaction and renew all available domains once in a while. If not, then I expect that one day the whole system is going to be completely broken. Are my concernsIs it correct or am I mistaken somewhere?
@alexvanin commented on GitHub (Oct 3, 2022):
@realloc @fyrchik Consider moving or creating new issue in the neofs-node to control renew process by Alphabet.
@alexvanin commented on GitHub (Nov 23, 2022):
containerzone is registered by container contract during deployment. Therefore admin of this zone is container contract itself. It means thatrenewmethod can be invoked only by container contract.Consider adding
Renewmethod in container contract that invokesrenewmethod in NNS./cc @fyrchik
@roman-khimov commented on GitHub (Dec 30, 2022):
We had this happen on mainnet (contract domain expiration, kills IR easily) and it required a manual fix, btw. The fix from #296 is just a ticking bomb for any installation.