Skip to content

Commit

Permalink
Merge pull request #2133 from CortexFoundation/dev
Browse files Browse the repository at this point in the history
cmd/devp2p: require dns:read, dns:edit permissions for cloudflare dep…
  • Loading branch information
ucwong authored Aug 29, 2024
2 parents c1744bf + 3137e60 commit c5f7d69
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion cmd/devp2p/dns_cloudflare.go
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,8 @@ func (c *cloudflareClient) checkZone(name string) error {
if !strings.HasSuffix(name, "."+zone.Name) {
return fmt.Errorf("CloudFlare zone name %q does not match name %q to be deployed", zone.Name, name)
}
needPerms := map[string]bool{"#zone:edit": false, "#zone:read": false}
// Necessary permissions for Cloudlare management - Zone:Read, DNS:Read, Zone:Edit, DNS:Edit
needPerms := map[string]bool{"#zone:edit": false, "#zone:read": false, "#dns_records:read": false, "#dns_records:edit": false}
for _, perm := range zone.Permissions {
if _, ok := needPerms[perm]; ok {
needPerms[perm] = true
Expand Down

0 comments on commit c5f7d69

Please sign in to comment.