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

   DSRs in versions earlier than 5.0 should have their requested _and_ original 
DS's set, but because of a bug in the downgrade process, the original was being 
discarded. This PR fixes that bug.
   
   <hr/>
   
   ## Which Traffic Control components are affected by this PR?
   - Traffic Ops
   
   ## What is the best way to verify this PR?
   I added unit test coverage for the case where neither original nor requested 
should be nil, so it should suffice that the tests pass. But the way this bug 
was found was because the DSR edit page was busted, so you may want to check 
that out to make sure it works.   
   
   ## If this is a bugfix, which Traffic Control versions contained the bug?
   - master
   
   ## PR submission checklist
   - [x] This PR has tests
   - [x] This PR doesn't need documentation
   - [x] This PR doesn't need 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to