Skip to content

Commit

Permalink
fix review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
forcodedancing committed Oct 17, 2023
1 parent b13a1e9 commit d6f0094
Showing 1 changed file with 0 additions and 12 deletions.
12 changes: 0 additions & 12 deletions x/permission/types/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -183,18 +183,6 @@ func (s *Statement) ValidateBasic(resType resource.ResourceType) error {
case resource.RESOURCE_TYPE_UNSPECIFIED:
return ErrInvalidStatement.Wrap("Please specify the ResourceType explicitly. Not allowed set RESOURCE_TYPE_UNSPECIFIED")
case resource.RESOURCE_TYPE_BUCKET:
//containsCreateObject := false
//for _, a := range s.Actions {
// if !BucketAllowedActions[a] {
// return ErrInvalidStatement.Wrapf("%s not allowed to be used on bucket.", a.String())
// }
// if a == ACTION_CREATE_OBJECT {
// containsCreateObject = true
// }
//}
//if !containsCreateObject && s.LimitSize != nil {
// return ErrInvalidStatement.Wrap("The LimitSize option can only be used with CreateObject actions at the bucket level. .")
//}
for _, r := range s.Resources {
var grn gnfd.GRN
err := grn.ParseFromString(r, true)
Expand Down

0 comments on commit d6f0094

Please sign in to comment.