[
https://issues.apache.org/jira/browse/TS-4459?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293558#comment-15293558
]
ASF GitHub Bot commented on TS-4459:
------------------------------------
Github user jpeach commented on the pull request:
https://github.com/apache/trafficserver/pull/647#issuecomment-220640925
I don't think that we should do the malloc and lowercasing on every lookup.
How about this ... when the caller indexes the certificate, check whether
the name is lowercased. If not, insert 2 entries; one lowercased and one with
the case provided. Then on the lookup you ought to hit one of the entries but
you don't care which one.
> Force domain names in cert to lower on insert into lookup tree
> --------------------------------------------------------------
>
> Key: TS-4459
> URL: https://issues.apache.org/jira/browse/TS-4459
> Project: Traffic Server
> Issue Type: Bug
> Components: SSL
> Reporter: Steven Feltner
> Fix For: 7.0.0
>
>
> We have certs from a legacy system that were issued with mixed case domain
> names. We are migrating this older product over to ATS and found that domain
> names need to be lower cased before being inserted in the lookup table.
> I will be submitting a pull request to resolve this issue.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
