Skip to content

Commit

Permalink
[RS-2325] Add rbac for waf api (#3746)
Browse files Browse the repository at this point in the history
* Add rbac for waf api

* lock down configmap perms to coreruleset
  • Loading branch information
LorcanMcVeigh authored Feb 4, 2025
1 parent 8841c2a commit aca5c78
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 0 deletions.
14 changes: 14 additions & 0 deletions pkg/render/apiserver.go
Original file line number Diff line number Diff line change
Expand Up @@ -1653,6 +1653,13 @@ func (c *apiServerComponent) tigeraUserClusterRole() *rbacv1.ClusterRole {
Resources: []string{"serviceaccounts"},
Verbs: []string{"list"},
},
// Access for WAF API to read in coreruleset configmap
{
APIGroups: []string{""},
Resources: []string{"configmaps"},
ResourceNames: []string{"coreruleset-default"},
Verbs: []string{"get"},
},
// Access to statistics.
{
APIGroups: []string{""},
Expand Down Expand Up @@ -1843,6 +1850,13 @@ func (c *apiServerComponent) tigeraNetworkAdminClusterRole() *rbacv1.ClusterRole
Resources: []string{"serviceaccounts"},
Verbs: []string{"list"},
},
// Access for WAF API to read in coreruleset configmap
{
APIGroups: []string{""},
Resources: []string{"configmaps"},
ResourceNames: []string{"coreruleset-default"},
Verbs: []string{"get"},
},
// Access to statistics.
{
APIGroups: []string{""},
Expand Down
12 changes: 12 additions & 0 deletions pkg/render/apiserver_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1369,6 +1369,12 @@ var (
Resources: []string{"serviceaccounts"},
Verbs: []string{"list"},
},
{
APIGroups: []string{""},
Resources: []string{"configmaps"},
ResourceNames: []string{"coreruleset-default"},
Verbs: []string{"get"},
},
{
APIGroups: []string{""},
Resources: []string{"services/proxy"},
Expand Down Expand Up @@ -1518,6 +1524,12 @@ var (
Resources: []string{"serviceaccounts"},
Verbs: []string{"list"},
},
{
APIGroups: []string{""},
Resources: []string{"configmaps"},
ResourceNames: []string{"coreruleset-default"},
Verbs: []string{"get"},
},
{
APIGroups: []string{""},
Resources: []string{"services/proxy"},
Expand Down

0 comments on commit aca5c78

Please sign in to comment.