[
https://issues.apache.org/jira/browse/TS-4108?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15308702#comment-15308702
]
ASF subversion and git services commented on TS-4108:
-----------------------------------------------------
Commit c185aff3d7b74916ac378615d3628234a638fcda in trafficserver's branch
refs/heads/6.2.x from [~jpe...@apache.org]
[ https://git-wip-us.apache.org/repos/asf?p=trafficserver.git;h=c185aff ]
TS-4108: Switch SSL metrics to RECD_COUNTER.
All the SSL metrics registered in SSLUtils.cc are specified as
RECD_INT, but they are all actually counters and should be specified
as RECD_COUNTER. They are counters because they all monotonically
increment and can be sensibly rate-converted.
(cherry picked from commit 5b5efae070169cb80061d1ec1d2c31799e096e43)
> SSL metrics have wrong data type
> --------------------------------
>
> Key: TS-4108
> URL: https://issues.apache.org/jira/browse/TS-4108
> Project: Traffic Server
> Issue Type: Bug
> Components: Metrics
> Reporter: James Peach
> Assignee: James Peach
> Fix For: 7.0.0
>
>
> All the SSL metrics registered in {{SSLUtils.cc}} are specified as
> {{RECD_INT}}, but they are all actually counters and should be specified as
> {{RECD_COUNTER}}.
> The sync function is sometimes given as {{RecRawStatSyncSum}} and sometimes
> {{RecRawStatSyncCount}}. Although these functions are identical, we should
> choose just one.
> Note that the SSL session cache metrics are a little quirky. They *are*
> actually counters, but they use {{SSL_SET_COUNT_DYN_STAT}} rather than
> {{SSL_INCREMENT_DYN_STAT}}. AFAICT the right way to do this would be to use
> {{SSL_INCREMENT_DYN_STAT}} but clear the {{SSL_CTX}} counters on every pass.
> The way it is done now seems like it would lead to incorrect values when the
> session cache is cleared or SSL state is reloaded. Unfortunately I don't see
> an OpenSSL API to clear the counters, so more research is needed there.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
