upd(system): add permission validation

crusttech / crust-server

Apache License 2.0

87 stars 21 forks source link

upd(system): add permission validation #48

Closed mitjaziv closed 5 years ago

mitjaziv commented 5 years ago

@titpetric @darh

Can we clarify if this two rules are now accepted as valid to save or no:

rules.Rule{Resource: "system", Operation: "organisation.create", Value: rules.Allow},
rules.Rule{Resource: "messaging:channel", Operation: "message.update_all", Value: rules.Allow}

If yes I need to change validation part a bit more.

darh commented 5 years ago

I would say yes - they are accepted as valid. Why not? Am I missing something?