Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Conversation

@ocket8888
Copy link
Contributor

This PR fixes #6603.


Which Traffic Control components are affected by this PR?

  • Traffic Ops (database)

What is the best way to verify this PR?

Create a Role with an "admin" "priv_level", then run the provided migration. Ensure the Role now has the DNS-SEC:READ and DNS-SEC:DELETE Permissions. Then run the "down" migration and ensure they no longer do (although that's probably much less important).

If this is a bugfix, which Traffic Control versions contained the bug?

  • master
  • 6.1.0

PR submission checklist

  • This PR has tests
  • This PR has documentation
  • This PR has a CHANGELOG.md entry
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

@ocket8888 ocket8888 added bug something isn't working as intended Traffic Ops related to Traffic Ops low impact affects only a small portion of a CDN, and cannot itself break one database relating to setup/installation/structure of the Traffic Ops database labels Mar 3, 2022
Copy link
Member

@shamrickus shamrickus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shamrickus shamrickus merged commit 28ee0e7 into apache:master Mar 3, 2022
@ocket8888 ocket8888 deleted the db/add-missing-perms branch March 3, 2022 17:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug something isn't working as intended database relating to setup/installation/structure of the Traffic Ops database low impact affects only a small portion of a CDN, and cannot itself break one Traffic Ops related to Traffic Ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing Permissions in DB seed/migration

2 participants