status-im
/
consul
mirror of https://github.com/status-im/consul.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Fixup dropped SecretID usage 

Looks like something got munged at some point. Not sure how it slipped in, but my best guess is that because TestTxn_Apply_ACLDeny is marked flaky we didn't block merge because it failed.

Signed-off-by: Mark Anderson <manderson@hashicorp.com>
This commit is contained in:
Mark Anderson 2022-03-22 20:41:13 -07:00
parent 4f1bfeaf33
commit 5590da2732
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
agent/consul/txn_endpoint_test.go
View File

@ -875,12 +875,11 @@ func TestTxn_Read_ACLDeny(t *testing.T) {
state.EnsureCheck(4, &check)
token := createTokenFull(t, codec, testTxnRules)
id := token.AccessorID
t.Run("simple read operations (results get filtered out)", func(t *testing.T) {
arg := structs.TxnReadRequest{
Datacenter: "dc1",
QueryOptions: structs.QueryOptions{Token: id},
QueryOptions: structs.QueryOptions{Token: token.SecretID},
Ops: structs.TxnOps{
{
KV: &structs.TxnKVOp{
@ -912,7 +911,7 @@ func TestTxn_Read_ACLDeny(t *testing.T) {
t.Run("complex operations (return permission denied errors)", func(t *testing.T) {
arg := structs.TxnReadRequest{
Datacenter: "dc1",
QueryOptions: structs.QueryOptions{Token: id},
QueryOptions: structs.QueryOptions{Token: token.SecretID},
Ops: structs.TxnOps{
{
KV: &structs.TxnKVOp{