Skip to content
Snippets Groups Projects
Commit f63fdaa5 authored by Thore Bödecker's avatar Thore Bödecker Committed by Ewoud Kohl van Wijngaarden
Browse files

also allow whitelisted admin clients to clean certs

During #728 a regression was introduced, denying the other whitelisted
admin clients cleaning/deletion of certificates:

```
2020-06-02T16:30:47.856+02:00 ERROR [qtp1105504743-114201] [p.t.a.rules] Forbidden request: puppetserver01.[...] access to /puppet-ca/v1/certificate_status/my.fancy.hostname (method :delete) (authenticated: true) denied by rule 'Allow nodes to delete their own certificates'.
```

The solution is to re-allow the entries within
`@server_admin_api_whitelist`, which usually contain "localhost" and the
fqdn of the puppetserver CA system.
parent 34d53f78
Branches master
No related tags found
No related merge requests found
...@@ -119,7 +119,17 @@ authorization: { ...@@ -119,7 +119,17 @@ authorization: {
type: regex type: regex
method: [delete] method: [delete]
}, },
allow: "$2" allow: [
"$2",
<%- @server_admin_api_whitelist.each do |client| -%>
"<%= client %>",
<%- end -%>
{
extensions: {
pp_cli_auth: "true"
}
}
]
sort-order: 500 sort-order: 500
}, },
<%- end -%> <%- end -%>
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment