[
https://issues.apache.org/jira/browse/TC-187?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16130655#comment-16130655
]
ASF GitHub Bot commented on TC-187:
-----------------------------------
Github user rawlinp commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/360#discussion_r133740451
--- Diff: traffic_ops/app/lib/UI/DeliveryService.pm ---
@@ -152,12 +153,7 @@ sub get_example_urls {
}
}
elsif ( $re->{type} eq 'PATH_REGEXP' ) {
- if ( defined( $example_urls[ $re->{set_number}
] ) ) {
- $example_urls[ $re->{set_number} ] .=
$re->{pattern};
- }
- else {
- $example_urls[ $re->{set_number} ] =
$re->{pattern};
- }
+ push(@example_urls, $re->{pattern});
--- End diff --
What made this change necessary? Doesn't the old code concatenate patterns
if they use the same `set_number`? This looks like it would just add multiple
different patterns using the same `set_number`s.
> Update delivery service makes SSL keys invalid
> ----------------------------------------------
>
> Key: TC-187
> URL: https://issues.apache.org/jira/browse/TC-187
> Project: Traffic Control
> Issue Type: Bug
> Components: Traffic Ops
> Reporter: Zhilin Huang
> Assignee: Zhilin Huang
> Labels: ssl, xml-id
>
> Modify xml-id or subdomain in a https delivery service will make existing SSL
> keys invalid.
> And there is problem to generate keys if using "http and https" together with
> 1 host_regex, and 1 path_regex.
> BTW, the certificate returned by RESTful API is not consistent with that GUI.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
