TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 106 Pull requests 12 Projects Releases 2 Activity Actions

[#999] morph: Use Global scope for proxy contract
All checks were successful
DCO action / DCO (pull_request) Successful in 2m28s
Details
Vulncheck / Vulncheck (pull_request) Successful in 2m52s
Details
Build / Build Components (1.21) (pull_request) Successful in 4m5s
Details
Build / Build Components (1.20) (pull_request) Successful in 4m12s
Details
Tests and linters / Lint (pull_request) Successful in 6m10s
Details
Tests and linters / Staticcheck (pull_request) Successful in 5m59s
Details
Tests and linters / Tests (1.20) (pull_request) Successful in 8m24s
Details
Tests and linters / Tests with -race (pull_request) Successful in 8m38s
Details
Tests and linters / Tests (1.21) (pull_request) Successful in 8m58s
Details

Browse source 
Proxy contract can now be used as an owner of NNS domains, thus we need
it not only to pay for the transaction but also to check domain
ownership. CalledByEntry is not enough, because we may register NNS
domains owned by proxy indirectly from the container contract.

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2024-02-20 18:59:35 +03:00
parent 7470c383dd
commit e18f0f5178
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
pkg/morph/client/notary.go
View file

@ -569,7 +569,11 @@ func (c *Client) notaryCosigners(invokedByAlpha bool, ir []*keys.PublicKey, comm
s[0] = actor.SignerAccount{
Signer: transaction.Signer{
Account: c.notary.proxy,
Scopes: transaction.None,
// Do not change this:
// We must be able to call NNS contract indirectly from the Container contract.
// Thus, CalledByEntry is not sufficient.
// In future we may restrict this to all the usecases we have.
Scopes: transaction.Global,
},
Account: notary.FakeContractAccount(c.notary.proxy),
}