You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2021/11/15 17:44:28 UTC

[GitHub] [trafficcontrol] ocket8888 opened a new pull request #6349: Fix admin_all_permission down script invalid column

ocket8888 opened a new pull request #6349:
URL: https://github.com/apache/trafficcontrol/pull/6349


   In #6324 the "down" script for the added migration has a bug where it specifies a (non-existent) column name instead of a string literal. This fixes that.
   
   <hr/>
   
   ## Which Traffic Control components are affected by this PR?
   - Traffic Ops
   
   ## What is the best way to verify this PR?
   1. Run all migrations
   2. Roll back at least to before the admin_all_permission migration.
   
   ## If this is a bugfix, which Traffic Control versions contained the bug?
   - master
   
   ## PR submission checklist
   - [ ] This PR has tests
   - [ ] This PR has documentation
   - [ ] This PR has a CHANGELOG.md entry
   - [x] This PR **DOES NOT FIX A SERIOUS SECURITY VULNERABILITY**


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] rawlinp merged pull request #6349: Fix admin_all_permission down script invalid column

Posted by GitBox <gi...@apache.org>.

rawlinp merged pull request #6349:
URL: https://github.com/apache/trafficcontrol/pull/6349


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org