TrueCloudLab/frostfs-contract
22
0
Fork 19
Code Issues 12 Pull requests 1 Projects Releases 2 Activity Actions

policy client: key is too big #54

New issue
Closed
opened 2023-11-23 11:36:30 +00:00 by dkirillov · 2 comments
dkirillov commented 2023-11-23 11:36:30 +00:00
Member
Copy link

I want the tests pass with the following patch to master

I want to add chain to policy contract with namespace root and unprintable name with length 38 (I expected key length in contract be 1 (prefix) + 2 (length namespace bytes) + 4 (namespace) + 38 (name) = 45)

It seems somewhere between client and neo-go we transform unprintable bytes to several bytes (38 became 60, 66 and some other lengths). If I try use neotest framework everything works as expected

diff --git a/policy/policy_contract.go b/policy/policy_contract.go
index 398468b..f2c8c94 100644
--- a/policy/policy_contract.go
+++ b/policy/policy_contract.go
@@ -3,6 +3,8 @@ package policy
 import (
        "git.frostfs.info/TrueCloudLab/frostfs-contract/common"
        "github.com/nspcc-dev/neo-go/pkg/interop/iterator"
+       "github.com/nspcc-dev/neo-go/pkg/interop/native/std"
+       "github.com/nspcc-dev/neo-go/pkg/interop/runtime"
        "github.com/nspcc-dev/neo-go/pkg/interop/storage"
 )
 
@@ -28,7 +30,8 @@ func storageKey(prefix Kind, entityName, name string) []byte {
 }
 
 func AddChain(entity Kind, entityName, name string, chain []byte) {
-       common.CheckAlphabetWitness() // TODO: Allow to work with chain directly for everyone?
+       runtime.Log(std.Itoa10(len(entityName)))
+       runtime.Log(std.Itoa10(len(name)))
 
        ctx := storage.GetContext()
        key := storageKey(entity, entityName, name)
diff --git a/tests/policy_test.go b/tests/policy_test.go
index d484d5a..b066317 100644
--- a/tests/policy_test.go
+++ b/tests/policy_test.go
@@ -2,14 +2,24 @@ package tests
 
 import (
        "bytes"
+       "context"
+       "crypto/md5"
+       "fmt"
+       "math/big"
        "path"
        "testing"
 
        "git.frostfs.info/TrueCloudLab/frostfs-contract/policy"
+       policycontract "git.frostfs.info/TrueCloudLab/frostfs-contract/policy"
+       policyclient "git.frostfs.info/TrueCloudLab/frostfs-contract/rpcclient/policy"
+       "github.com/nspcc-dev/neo-go/pkg/crypto/keys"
        "github.com/nspcc-dev/neo-go/pkg/neotest"
+       "github.com/nspcc-dev/neo-go/pkg/rpcclient"
+       "github.com/nspcc-dev/neo-go/pkg/rpcclient/actor"
        "github.com/nspcc-dev/neo-go/pkg/util"
        "github.com/nspcc-dev/neo-go/pkg/vm"
        "github.com/nspcc-dev/neo-go/pkg/vm/stackitem"
+       "github.com/nspcc-dev/neo-go/pkg/wallet"
        "github.com/stretchr/testify/require"
 )
 
@@ -28,6 +38,48 @@ func newPolicyInvoker(t *testing.T) *neotest.ContractInvoker {
        return e.CommitteeInvoker(h)
 }
 
+func TestBigKey(t *testing.T) {
+       e := newPolicyInvoker(t)
+
+       acc, err := wallet.NewAccount()
+       require.NoError(t, err)
+
+       wlt := initTmpWallet(t)
+       ctx, cancel := context.WithCancel(context.Background())
+       defer cancel()
+       address := runRPC(ctx, t, e.Chain, wlt)
+
+       rpcCli, err := rpcclient.New(context.Background(), "http://"+address, rpcclient.Options{})
+       require.NoError(t, err)
+
+       act, err := actor.NewSimple(rpcCli, acc)
+       require.NoError(t, err)
+
+       cli := policyclient.New(act, e.Hash)
+
+       a := awaiter{
+               ctx: ctx,
+               t:   t,
+               rpc: rpcCli,
+       }
+
+       p1 := []byte("chain1")
+       key, err := keys.NewPrivateKey()
+       require.NoError(t, err)
+
+       hash := md5.New()
+       hash.Write([]byte("policy"))
+       suffix := hash.Sum(nil)
+
+       ns := "root"
+       fmt.Println(len(ns))
+       name := "i" + string(key.PublicKey().GetScriptHash().BytesBE()) + "/" + string(suffix)
+       fmt.Println(len(name))
+
+       a.await(cli.AddChain(big.NewInt(policycontract.IAM), ns, name, p1))
+
+}
+

test output:

=== RUN   TestBigKey
    logger.go:130: 2023-11-23T14:38:06.107+0300	INFO	initial gas supply is not set or wrong, setting default value	{"InitialGASSupply": "52000000"}
    logger.go:130: 2023-11-23T14:38:06.107+0300	INFO	mempool size is not set or wrong, setting default value	{"MemPoolSize": 50000}
    logger.go:130: 2023-11-23T14:38:06.107+0300	INFO	MaxBlockSize is not set or wrong, setting default value	{"MaxBlockSize": 262144}
    logger.go:130: 2023-11-23T14:38:06.107+0300	INFO	MaxBlockSystemFee is not set or wrong, setting default value	{"MaxBlockSystemFee": 900000000000}
    logger.go:130: 2023-11-23T14:38:06.107+0300	INFO	MaxTransactionsPerBlock is not set or wrong, using default value	{"MaxTransactionsPerBlock": 512}
    logger.go:130: 2023-11-23T14:38:06.107+0300	INFO	MaxValidUntilBlockIncrement is not set or wrong, using default value	{"MaxValidUntilBlockIncrement": 86400}
    logger.go:130: 2023-11-23T14:38:06.107+0300	INFO	NativeActivations are not set, using default values
    logger.go:130: 2023-11-23T14:38:06.107+0300	INFO	Hardforks are not set, using default value
    logger.go:130: 2023-11-23T14:38:06.110+0300	INFO	no storage version found! creating genesis block
    logger.go:130: 2023-11-23T14:38:06.177+0300	DEBUG	done processing headers	{"headerIndex": 1, "blockHeight": 0, "took": "176.272µs"}
{"level":"info","msg":"ExtensiblePoolSize is not set or wrong, using default value","ExtensiblePoolSize":20}
{"level":"info","msg":"bad MaxPeers configured, using the default value","configured":0,"actual":100}
{"level":"info","msg":"bad AttemptConnPeers configured, using the default value","configured":0,"actual":20}
{"level":"info","msg":"MaxFindResultItems is not set or wrong, setting default value","MaxFindResultItems":100}
{"level":"info","msg":"MaxFindStorageResultItems is not set or wrong, setting default value","MaxFindStorageResultItems":50}
{"level":"info","msg":"MaxNEP11Tokens is not set or wrong, setting default value","MaxNEP11Tokens":100}
{"level":"info","msg":"MaxWebSocketClients is not set or wrong, setting default value","MaxWebSocketClients":64}
{"level":"info","msg":"node started","blockHeight":1,"headerHeight":1}
{"level":"info","msg":"node reached synchronized state, starting services"}
{"level":"info","msg":"starting state validation service"}
{"level":"info","msg":"starting consensus service"}
{"level":"info","msg":"initializing dbft","height":2,"view":0,"index":0,"role":"Primary"}
{"level":"debug","msg":"reset timer","h":2,"v":0,"delay":"100ms"}
{"level":"debug","msg":"broadcasting message","type":"PrepareRequest","height":2,"view":0}
{"level":"info","msg":"sending PrepareRequest","height":2,"view":0}
{"level":"debug","msg":"reset timer","h":2,"v":0,"delay":"100ms"}
{"level":"debug","msg":"check preparations","hasReq":true,"count":1,"M":1}
{"level":"info","msg":"sending Commit","height":2,"view":0}
{"level":"debug","msg":"broadcasting message","type":"Commit","height":2,"view":0}
{"level":"debug","msg":"reset timer","h":2,"v":0,"delay":"100ms"}
{"level":"info","msg":"approving block","height":2,"hash":"6381136c686cbae57534aec235f899ea26566297ff93d1505b56b919cc0633be","tx_count":0,"merkle":"0000000000000000000000000000000000000000000000000000000000000000","prev":"6bb78eb4443c8eef89759f7710ec0f183d4e4f287631ab1ba7298cba402750cf"}
{"level":"info","msg":"starting rpc-server","endpoint":"localhost:0"}
    logger.go:130: 2023-11-23T14:38:06.596+0300	DEBUG	done processing headers	{"headerIndex": 2, "blockHeight": 1, "took": "189.106µs"}
{"level":"debug","msg":"new block in the chain","dbft index":2,"chain index":2}
{"level":"info","msg":"initializing dbft","height":3,"view":0,"index":0,"role":"Primary"}
{"level":"debug","msg":"reset timer","h":3,"v":0,"delay":"98.902665ms"}
    util.go:221: RPC server is listening at 127.0.0.1:46789, checking health
{"level":"info","msg":"Error encountered with rpc request","code":-32602,"cause":"invalid method 'GET', please retry with 'POST'","method":"","params":"[]"}
{"level":"debug","msg":"timer fired","height":3,"view":0}
{"level":"debug","msg":"timeout","height":3,"view":0}
{"level":"debug","msg":"broadcasting message","type":"PrepareRequest","height":3,"view":0}
{"level":"info","msg":"sending PrepareRequest","height":3,"view":0}
{"level":"debug","msg":"reset timer","h":3,"v":0,"delay":"100ms"}
{"level":"debug","msg":"check preparations","hasReq":true,"count":1,"M":1}
{"level":"info","msg":"sending Commit","height":3,"view":0}
{"level":"debug","msg":"broadcasting message","type":"Commit","height":3,"view":0}
{"level":"debug","msg":"reset timer","h":3,"v":0,"delay":"100ms"}
{"level":"info","msg":"approving block","height":3,"hash":"0034a3254b33fa84fac4619f779a239f7ac886d8b7fba6d394db2d8d025f5ef2","tx_count":0,"merkle":"0000000000000000000000000000000000000000000000000000000000000000","prev":"6381136c686cbae57534aec235f899ea26566297ff93d1505b56b919cc0633be"}
{"level":"debug","msg":"processing rpc request","method":"getversion","params":"[]"}
    logger.go:130: 2023-11-23T14:38:06.696+0300	DEBUG	done processing headers	{"headerIndex": 3, "blockHeight": 2, "took": "106.791µs"}
{"level":"debug","msg":"new block in the chain","dbft index":3,"chain index":3}
{"level":"info","msg":"initializing dbft","height":4,"view":0,"index":0,"role":"Primary"}
{"level":"debug","msg":"reset timer","h":4,"v":0,"delay":"99.667153ms"}
4
38
{"level":"debug","msg":"processing rpc request","method":"invokefunction","params":"[ca3d1d58d8b9f61da1500b2e416bb114b5fe4374 addChain  ]"}
    logger.go:130: 2023-11-23T14:38:06.697+0300	INFO	runtime log	{"tx": "de22aea3da305ec2f52f2e2c16f39d8ef961a4f711380584c1376fceb5f3bc0a", "script": "ca3d1d58d8b9f61da1500b2e416bb114b5fe4374", "msg": "4"}
    logger.go:130: 2023-11-23T14:38:06.697+0300	INFO	runtime log	{"tx": "de22aea3da305ec2f52f2e2c16f39d8ef961a4f711380584c1376fceb5f3bc0a", "script": "ca3d1d58d8b9f61da1500b2e416bb114b5fe4374", "msg": "60"}
{"level":"debug","msg":"processing rpc request","method":"getblockcount","params":"[]"}
{"level":"debug","msg":"processing rpc request","method":"calculatenetworkfee","params":"[AAKfhBFMqHQAAAAAAAAAAAAAAAAABgAAAAGro4Padf8t06CliR+zqOlg9fT32AEAdgwGY2hhaW4xDDxp77+9I0JtKTTGou+/vR8aLTvvv73NhVJm77+9ci/vv73vv73vv71X77+9V2wADO+/vRBbJWDvv73vv70MBHJvb3QAaRTAHwwIYWRkQ2hhaW4MFHRD/rUUsWtBLgtQoR32udhYHT3KQWJ9W1IBACgMIQK/5/e/22kcwn9EkybSpiOrzfF4tN7/Rw0gW3UnSRXFL0FW57Mn]"}
    util.go:37: 
        	Error Trace:	/home/denis/github/tcl/neofs-contract/tests/util.go:37
        	            				/home/denis/github/tcl/neofs-contract/tests/policy_test.go:79
        	Error:      	Received unexpected error:
        	            	script failed (FAULT state) due to an error: at instruction 268 (SYSCALL): System.Storage.Put failed: key is too big
        	Test:       	TestBigKey
{"level":"info","msg":"shutting down server","peers":0}
{"level":"info","msg":"stopping state validation service"}
{"level":"info","msg":"stopping consensus service"}
{"level":"info","msg":"shutting down RPC server","endpoint":"127.0.0.1:46789"}
    logger.go:130: 2023-11-23T14:38:06.698+0300	INFO	persisted to disk	{"blocks": 3, "keys": 170, "headerHeight": 3, "blockHeight": 3, "took": "68.428µs"}
--- FAIL: TestBigKey (0.59s)

FAIL

Process finished with the exit code 1
I want the tests pass with the following patch to [master](https://git.frostfs.info/TrueCloudLab/frostfs-contract/commit/20f86e96b27c3e42677ed6cc06cac3152f86a98f) I want to add chain to `policy` contract with namespace `root` and unprintable name with length `38` (I expected key length in contract be `1 (prefix) + 2 (length namespace bytes) + 4 (namespace) + 38 (name) = 45`) It seems somewhere between client and neo-go we transform unprintable bytes to several bytes (38 became 60, 66 and some other lengths). If I try use neotest framework everything works as expected ```diff diff --git a/policy/policy_contract.go b/policy/policy_contract.go index 398468b..f2c8c94 100644 --- a/policy/policy_contract.go +++ b/policy/policy_contract.go @@ -3,6 +3,8 @@ package policy import ( "git.frostfs.info/TrueCloudLab/frostfs-contract/common" "github.com/nspcc-dev/neo-go/pkg/interop/iterator" + "github.com/nspcc-dev/neo-go/pkg/interop/native/std" + "github.com/nspcc-dev/neo-go/pkg/interop/runtime" "github.com/nspcc-dev/neo-go/pkg/interop/storage" ) @@ -28,7 +30,8 @@ func storageKey(prefix Kind, entityName, name string) []byte { } func AddChain(entity Kind, entityName, name string, chain []byte) { - common.CheckAlphabetWitness() // TODO: Allow to work with chain directly for everyone? + runtime.Log(std.Itoa10(len(entityName))) + runtime.Log(std.Itoa10(len(name))) ctx := storage.GetContext() key := storageKey(entity, entityName, name) diff --git a/tests/policy_test.go b/tests/policy_test.go index d484d5a..b066317 100644 --- a/tests/policy_test.go +++ b/tests/policy_test.go @@ -2,14 +2,24 @@ package tests import ( "bytes" + "context" + "crypto/md5" + "fmt" + "math/big" "path" "testing" "git.frostfs.info/TrueCloudLab/frostfs-contract/policy" + policycontract "git.frostfs.info/TrueCloudLab/frostfs-contract/policy" + policyclient "git.frostfs.info/TrueCloudLab/frostfs-contract/rpcclient/policy" + "github.com/nspcc-dev/neo-go/pkg/crypto/keys" "github.com/nspcc-dev/neo-go/pkg/neotest" + "github.com/nspcc-dev/neo-go/pkg/rpcclient" + "github.com/nspcc-dev/neo-go/pkg/rpcclient/actor" "github.com/nspcc-dev/neo-go/pkg/util" "github.com/nspcc-dev/neo-go/pkg/vm" "github.com/nspcc-dev/neo-go/pkg/vm/stackitem" + "github.com/nspcc-dev/neo-go/pkg/wallet" "github.com/stretchr/testify/require" ) @@ -28,6 +38,48 @@ func newPolicyInvoker(t *testing.T) *neotest.ContractInvoker { return e.CommitteeInvoker(h) } +func TestBigKey(t *testing.T) { + e := newPolicyInvoker(t) + + acc, err := wallet.NewAccount() + require.NoError(t, err) + + wlt := initTmpWallet(t) + ctx, cancel := context.WithCancel(context.Background()) + defer cancel() + address := runRPC(ctx, t, e.Chain, wlt) + + rpcCli, err := rpcclient.New(context.Background(), "http://"+address, rpcclient.Options{}) + require.NoError(t, err) + + act, err := actor.NewSimple(rpcCli, acc) + require.NoError(t, err) + + cli := policyclient.New(act, e.Hash) + + a := awaiter{ + ctx: ctx, + t: t, + rpc: rpcCli, + } + + p1 := []byte("chain1") + key, err := keys.NewPrivateKey() + require.NoError(t, err) + + hash := md5.New() + hash.Write([]byte("policy")) + suffix := hash.Sum(nil) + + ns := "root" + fmt.Println(len(ns)) + name := "i" + string(key.PublicKey().GetScriptHash().BytesBE()) + "/" + string(suffix) + fmt.Println(len(name)) + + a.await(cli.AddChain(big.NewInt(policycontract.IAM), ns, name, p1)) + +} + ``` test output: ``` === RUN TestBigKey logger.go:130: 2023-11-23T14:38:06.107+0300 INFO initial gas supply is not set or wrong, setting default value {"InitialGASSupply": "52000000"} logger.go:130: 2023-11-23T14:38:06.107+0300 INFO mempool size is not set or wrong, setting default value {"MemPoolSize": 50000} logger.go:130: 2023-11-23T14:38:06.107+0300 INFO MaxBlockSize is not set or wrong, setting default value {"MaxBlockSize": 262144} logger.go:130: 2023-11-23T14:38:06.107+0300 INFO MaxBlockSystemFee is not set or wrong, setting default value {"MaxBlockSystemFee": 900000000000} logger.go:130: 2023-11-23T14:38:06.107+0300 INFO MaxTransactionsPerBlock is not set or wrong, using default value {"MaxTransactionsPerBlock": 512} logger.go:130: 2023-11-23T14:38:06.107+0300 INFO MaxValidUntilBlockIncrement is not set or wrong, using default value {"MaxValidUntilBlockIncrement": 86400} logger.go:130: 2023-11-23T14:38:06.107+0300 INFO NativeActivations are not set, using default values logger.go:130: 2023-11-23T14:38:06.107+0300 INFO Hardforks are not set, using default value logger.go:130: 2023-11-23T14:38:06.110+0300 INFO no storage version found! creating genesis block logger.go:130: 2023-11-23T14:38:06.177+0300 DEBUG done processing headers {"headerIndex": 1, "blockHeight": 0, "took": "176.272µs"} {"level":"info","msg":"ExtensiblePoolSize is not set or wrong, using default value","ExtensiblePoolSize":20} {"level":"info","msg":"bad MaxPeers configured, using the default value","configured":0,"actual":100} {"level":"info","msg":"bad AttemptConnPeers configured, using the default value","configured":0,"actual":20} {"level":"info","msg":"MaxFindResultItems is not set or wrong, setting default value","MaxFindResultItems":100} {"level":"info","msg":"MaxFindStorageResultItems is not set or wrong, setting default value","MaxFindStorageResultItems":50} {"level":"info","msg":"MaxNEP11Tokens is not set or wrong, setting default value","MaxNEP11Tokens":100} {"level":"info","msg":"MaxWebSocketClients is not set or wrong, setting default value","MaxWebSocketClients":64} {"level":"info","msg":"node started","blockHeight":1,"headerHeight":1} {"level":"info","msg":"node reached synchronized state, starting services"} {"level":"info","msg":"starting state validation service"} {"level":"info","msg":"starting consensus service"} {"level":"info","msg":"initializing dbft","height":2,"view":0,"index":0,"role":"Primary"} {"level":"debug","msg":"reset timer","h":2,"v":0,"delay":"100ms"} {"level":"debug","msg":"broadcasting message","type":"PrepareRequest","height":2,"view":0} {"level":"info","msg":"sending PrepareRequest","height":2,"view":0} {"level":"debug","msg":"reset timer","h":2,"v":0,"delay":"100ms"} {"level":"debug","msg":"check preparations","hasReq":true,"count":1,"M":1} {"level":"info","msg":"sending Commit","height":2,"view":0} {"level":"debug","msg":"broadcasting message","type":"Commit","height":2,"view":0} {"level":"debug","msg":"reset timer","h":2,"v":0,"delay":"100ms"} {"level":"info","msg":"approving block","height":2,"hash":"6381136c686cbae57534aec235f899ea26566297ff93d1505b56b919cc0633be","tx_count":0,"merkle":"0000000000000000000000000000000000000000000000000000000000000000","prev":"6bb78eb4443c8eef89759f7710ec0f183d4e4f287631ab1ba7298cba402750cf"} {"level":"info","msg":"starting rpc-server","endpoint":"localhost:0"} logger.go:130: 2023-11-23T14:38:06.596+0300 DEBUG done processing headers {"headerIndex": 2, "blockHeight": 1, "took": "189.106µs"} {"level":"debug","msg":"new block in the chain","dbft index":2,"chain index":2} {"level":"info","msg":"initializing dbft","height":3,"view":0,"index":0,"role":"Primary"} {"level":"debug","msg":"reset timer","h":3,"v":0,"delay":"98.902665ms"} util.go:221: RPC server is listening at 127.0.0.1:46789, checking health {"level":"info","msg":"Error encountered with rpc request","code":-32602,"cause":"invalid method 'GET', please retry with 'POST'","method":"","params":"[]"} {"level":"debug","msg":"timer fired","height":3,"view":0} {"level":"debug","msg":"timeout","height":3,"view":0} {"level":"debug","msg":"broadcasting message","type":"PrepareRequest","height":3,"view":0} {"level":"info","msg":"sending PrepareRequest","height":3,"view":0} {"level":"debug","msg":"reset timer","h":3,"v":0,"delay":"100ms"} {"level":"debug","msg":"check preparations","hasReq":true,"count":1,"M":1} {"level":"info","msg":"sending Commit","height":3,"view":0} {"level":"debug","msg":"broadcasting message","type":"Commit","height":3,"view":0} {"level":"debug","msg":"reset timer","h":3,"v":0,"delay":"100ms"} {"level":"info","msg":"approving block","height":3,"hash":"0034a3254b33fa84fac4619f779a239f7ac886d8b7fba6d394db2d8d025f5ef2","tx_count":0,"merkle":"0000000000000000000000000000000000000000000000000000000000000000","prev":"6381136c686cbae57534aec235f899ea26566297ff93d1505b56b919cc0633be"} {"level":"debug","msg":"processing rpc request","method":"getversion","params":"[]"} logger.go:130: 2023-11-23T14:38:06.696+0300 DEBUG done processing headers {"headerIndex": 3, "blockHeight": 2, "took": "106.791µs"} {"level":"debug","msg":"new block in the chain","dbft index":3,"chain index":3} {"level":"info","msg":"initializing dbft","height":4,"view":0,"index":0,"role":"Primary"} {"level":"debug","msg":"reset timer","h":4,"v":0,"delay":"99.667153ms"} 4 38 {"level":"debug","msg":"processing rpc request","method":"invokefunction","params":"[ca3d1d58d8b9f61da1500b2e416bb114b5fe4374 addChain ]"} logger.go:130: 2023-11-23T14:38:06.697+0300 INFO runtime log {"tx": "de22aea3da305ec2f52f2e2c16f39d8ef961a4f711380584c1376fceb5f3bc0a", "script": "ca3d1d58d8b9f61da1500b2e416bb114b5fe4374", "msg": "4"} logger.go:130: 2023-11-23T14:38:06.697+0300 INFO runtime log {"tx": "de22aea3da305ec2f52f2e2c16f39d8ef961a4f711380584c1376fceb5f3bc0a", "script": "ca3d1d58d8b9f61da1500b2e416bb114b5fe4374", "msg": "60"} {"level":"debug","msg":"processing rpc request","method":"getblockcount","params":"[]"} {"level":"debug","msg":"processing rpc request","method":"calculatenetworkfee","params":"[AAKfhBFMqHQAAAAAAAAAAAAAAAAABgAAAAGro4Padf8t06CliR+zqOlg9fT32AEAdgwGY2hhaW4xDDxp77+9I0JtKTTGou+/vR8aLTvvv73NhVJm77+9ci/vv73vv73vv71X77+9V2wADO+/vRBbJWDvv73vv70MBHJvb3QAaRTAHwwIYWRkQ2hhaW4MFHRD/rUUsWtBLgtQoR32udhYHT3KQWJ9W1IBACgMIQK/5/e/22kcwn9EkybSpiOrzfF4tN7/Rw0gW3UnSRXFL0FW57Mn]"} util.go:37: Error Trace: /home/denis/github/tcl/neofs-contract/tests/util.go:37 /home/denis/github/tcl/neofs-contract/tests/policy_test.go:79 Error: Received unexpected error: script failed (FAULT state) due to an error: at instruction 268 (SYSCALL): System.Storage.Put failed: key is too big Test: TestBigKey {"level":"info","msg":"shutting down server","peers":0} {"level":"info","msg":"stopping state validation service"} {"level":"info","msg":"stopping consensus service"} {"level":"info","msg":"shutting down RPC server","endpoint":"127.0.0.1:46789"} logger.go:130: 2023-11-23T14:38:06.698+0300 INFO persisted to disk {"blocks": 3, "keys": 170, "headerHeight": 3, "blockHeight": 3, "took": "68.428µs"} --- FAIL: TestBigKey (0.59s) FAIL Process finished with the exit code 1 ```
dkirillov added the
bug
 label 2023-11-23 11:36:30 +00:00
dkirillov changed title from policy client: keys too big to policy client: key is too big 2023-11-23 11:38:23 +00:00
fyrchik commented 2023-11-23 11:53:37 +00:00
Owner
Copy link

The problem most likely is non-UTF8 json marshaling/unmarshaling (used for RPC)
Let's change everything to bytes then, though it would be nice to have user friendly ID here. Is it possible, or do we hit a key limit?

The problem most likely is non-UTF8 json marshaling/unmarshaling (used for RPC) Let's change everything to bytes then, though it would be nice to have user friendly ID here. Is it possible, or do we hit a key limit?
dkirillov commented 2023-11-27 12:29:23 +00:00
Author
Member
Copy link

Is it possible, or do we hit a key limit?

Yes, we hit key limit since we need the key to consist of namespace, user key derivations and policy name

> Is it possible, or do we hit a key limit? Yes, we hit key limit since we need the key to consist of namespace, user key derivations and policy name
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
kek
support/v0.19
support/v0.18
support/v0.17
support/v0.16
v0.14.1-hotfix
enable-notary-in-public-chains
v0.21.0-rc.4
v0.21.0-rc.3
v0.21.0-rc.2
v0.21.0-rc.1
v0.20.0
v0.19.4
v0.19.3
v0.19.2
v0.19.1
v0.19.0
v0.18.0
Labels
Clear labels   
P0

Highest

  
P1

High

  
P2

Medium

  
P3

Low

  
good first issue

Good for newcomers

  
Infrastructure

Infrastructure, Automation, CI/CD or DevOps related stuff

  
blocked

The issue or PR is blocked by something

  
bug

Something is not working

  
config

Configuration format update or breaking change

  
discussion

Talking about something in order to reach a decision or to exchange ideas

  
documentation

Documentation related

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
go

Language features adoption

  
help wanted

Extra attention is needed

  
internal

   
invalid

Something is wrong

  
kludge

This is a kludge and we know it

  
observability

Related to metrics, tracing and logs

  
perfomance

Perfomance related

  
question

More information is needed

  
refactoring

Refactoring

  
wontfix

This won't be fixed

No labels
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
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-contract#54
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?