[GitHub] [trafficcontrol] shamrickus opened a new issue #6348: traffic_vault_migrate does not handle `-` correctly.

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

shamrickus opened a new issue #6348:
URL: https://github.com/apache/trafficcontrol/issues/6348


   <!--
   ************ STOP!! ************
   If this issue identifies a security vulnerability, DO NOT submit it! Instead, contact
   the Apache Traffic Control Security Team at security@trafficcontrol.apache.org and follow the
   guidelines at https://apache.org/security regarding vulnerability disclosure.
   
   - For *SUPPORT QUESTIONS*, use the #traffic-control channel on the ASF slack (https://s.apache.org/tc-slack-request)
   or the Traffic Control Users mailing list (send an email to users-subscribe@trafficcontrol.apache.org to subscribe).
   - Before submitting, please **SEARCH GITHUB** for a similar issue or PR
       * https://github.com/apache/trafficcontrol/issues
       * https://github.com/apache/trafficcontrol/pulls
   -->
   
   <!-- Do not submit security vulnerabilities or support requests here - see above -->
   ## This Bug Report affects these Traffic Control components:
   <!-- delete all those that don't apply -->
   - Tools
   
   ## Current behavior:
   If a CDN has a `-` in it's name, the SSL key version will be incorrectly extracted from the Riak database; which will then insert it into the destination DB without error (excluding duplicates which is a very common case).
   
   <!-- Describe how the bug happens -->
   
   
   ## Expected behavior:
   All valid CDN/DS names should be supported without issue.
   <!-- Describe what the behavior would be without the bug -->
   
   
   ## Steps to reproduce:
   While using Riak, create CDN with a `-` in the name, create a corresponding DS with SSL keys. Run the `traffic_vault_migrate` tool which will then insert the bad version data into postgres if there are no duplicates.
   <!-- If the current behavior is a bug, please provide the *STEPS TO REPRODUCE* and
   include the applicable TC version.
   -->
   
   


-- 
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] mattjackson220 closed issue #6348: traffic_vault_migrate does not handle `-` correctly.

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

mattjackson220 closed issue #6348:
URL: https://github.com/apache/trafficcontrol/issues/6348


   


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