luxolus/open-vault
2
0
Fork 0
Code Issues 1 Pull requests Releases Activity

Use logical operations instead of strings for comparison

Browse source
This commit is contained in:
Jeff Mitchell 2016-01-12 21:16:31 -05:00
parent d949043cac
commit f9bbe0fb04
2 changed files with 7 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
logical/request.go
View file

@ -134,10 +134,6 @@ const (
RollbackOperation = "rollback" RollbackOperation = "rollback"
) )
func (o Operation) String() string {
return string(o)
}
var ( var (
// ErrUnsupportedOperation is returned if the operation is not supported // ErrUnsupportedOperation is returned if the operation is not supported
// by the logical backend. // by the logical backend.

14
vault/acl.go
View file

@ -107,20 +107,20 @@ CHECK:
// If "deny" has been explicitly set, only deny will be in the map, so we // If "deny" has been explicitly set, only deny will be in the map, so we
// only need to check for the existence of other values // only need to check for the existence of other values
sudo = capabilities&SudoCapabilityInt > 0 sudo = capabilities&SudoCapabilityInt > 0
switch op.String() { switch op {
case "read": case logical.ReadOperation:
allowed = capabilities&ReadCapabilityInt > 0 allowed = capabilities&ReadCapabilityInt > 0
case "list": case logical.ListOperation:
allowed = capabilities&ListCapabilityInt > 0 allowed = capabilities&ListCapabilityInt > 0
case "update": case logical.UpdateOperation:
allowed = capabilities&UpdateCapabilityInt > 0 allowed = capabilities&UpdateCapabilityInt > 0
case "delete": case logical.DeleteOperation:
allowed = capabilities&DeleteCapabilityInt > 0 allowed = capabilities&DeleteCapabilityInt > 0
case "create": case logical.CreateOperation:
allowed = capabilities&CreateCapabilityInt > 0 allowed = capabilities&CreateCapabilityInt > 0
// These three re-use UpdateCapabilityInt since that's the most appropraite capability/operation mapping // These three re-use UpdateCapabilityInt since that's the most appropraite capability/operation mapping
case "revoke", "renew", "rollback": case logical.RevokeOperation, logical.RenewOperation, logical.RollbackOperation:
allowed = capabilities&UpdateCapabilityInt > 0 allowed = capabilities&UpdateCapabilityInt > 0
default: default: