Skip to content

Commit 70265ff

Browse files
committed
Re-set role GID to 2 in EOS AddACL
1 parent 560821b commit 70265ff

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
Bugfix: fix missing GID 2 in EOS AddACL
2+
3+
https://github.com/cs3org/reva/pull/5584

pkg/storage/fs/eos/client/grpc/acl.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ func (c *Client) AddACL(ctx context.Context, auth eosclient.Authorization, path
3636
return err
3737
}
3838

39+
rq.Role.Gid = 2
40+
3941
msg := new(erpc.NSRequest_AclRequest)
4042
msg.Cmd = erpc.NSRequest_AclRequest_ACL_COMMAND(erpc.NSRequest_AclRequest_ACL_COMMAND_value["MODIFY"])
4143
msg.Type = erpc.NSRequest_AclRequest_ACL_TYPE(erpc.NSRequest_AclRequest_ACL_TYPE_value["SYS_ACL"])
@@ -59,7 +61,7 @@ func (c *Client) AddACL(ctx context.Context, auth eosclient.Authorization, path
5961
return errtypes.NotFound(fmt.Sprintf("Path: %s", path))
6062
}
6163

62-
log.Debug().Str("func", "AddACL").Str("path", path).Str("resp:", fmt.Sprintf("%#v", resp)).Msg("grpc response")
64+
log.Debug().Str("func", "AddACL").Str("path", path).Str("resp:", fmt.Sprintf("%#v", resp)).Any("acl", resp.Acl).Any("error", resp.Error).Msg("grpc response")
6365

6466
return err
6567
}

0 commit comments

Comments
 (0)