Replacing ccr during config file generation #1807
Conversation
- Allow CNAME for hostname with `ccr`
|
Refer to this link for build results (access rights to CI server needed): |
| my $portstr = ""; | ||
| if ( $hname eq "ccr" && $server_obj->tcp_port > 0 && $server_obj->tcp_port != 80 ) { | ||
| if ( $hname eq "__http__r" && $server_obj->tcp_port > 0 && $server_obj->tcp_port != 80 ) { |
There was a problem hiding this comment.
@smalenfant - this looks like a typo? __http__r?
|
What about traffic_ops/app/lib/UI/ConfigFiles.pm? I think you had that in your original PR but missed it in this one? |
|
@mitchell852 I'll fix that. @rawlinp I can put |
- fix typo in ApacheTrafficServer.pm related to __http__r - Added ConfigFiles.pm for backward compatilibity
|
Refer to this link for build results (access rights to CI server needed): |
|
@smalenfant yeah, I think until we officially deprecate the TO UI we should try to keep parity. This looks good to me 👍 |
ccrby replacing the regex