[GitHub] [trafficcontrol] ocket8888 opened a new pull request #6444: Remove unnecessary logging of SSL cert/key paths

[GitBox <gi...@apache.org>]

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


   Currently, if `cdn.conf` specifies cert/key paths that don't match the values supplied to postinstall, it will log all four specified SSL key/cert paths. This is a bit unnecessary, since anyone with the permissions to be setting those values usefully will already have access to all four, and we need not clutter the logging output with it.
   
   <hr/>
   
   ## Which Traffic Control components are affected by this PR?
   - Traffic Ops (postinstall)
   
   ## What is the best way to verify this PR?
   Make sure postinstall tests still pass, make a Traffic Ops config then run postinstall with mismatched values for the ssl key/cert paths and make sure they don't get logged.
   
   ## 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] zrhoffman merged pull request #6444: Remove unnecessary logging of SSL cert/key paths

[GitBox <gi...@apache.org>]

zrhoffman merged pull request #6444:
URL: https://github.com/apache/trafficcontrol/pull/6444


   


-- 
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