intelsdi-x / snap-plugin-collector-apache Goto Github PK

If SSLSessionCache is enabled in Apache config, mod_status returns extra entries (even if mod_status isn't fetched over SSL), which cause the plugin to produce schema errors and fail. The extra entries are:

CacheAvgEntrySize
CacheCurrentEntries
CacheCurrentSize
CacheMaximumSize
CacheType

The errors are of the form:

time="2017-02-25T02:44:28Z" level=error msg="collector run error" _module=scheduler-job block=run error="rpc error: code = 2 desc = schema: invalid path \"CacheCurrentEntries\" (and 4 other errors)" job-type=collector

Username: Apache username
Password: Apache password
Verify Peer: Enable or disable peer SSL certificate verification
Verify Host: Enable or disable host name verification
CA Certificate file
SSLCiphers
Timeout

What happened:
Binary names in documentation are outdated

What you expected to happen:
Update snap binary names to snaptel/snapteld

Right now there needs to be a separate task for each instance of the Apache web server. Code can be refactored to allow multiple instances in one task if there is a demand for this. Commentary wanted.

Relates to #21.

There are some additional metrics related to TLS Session Caching that should be added, such as:

CacheAvgEntrySize
CacheCurrentEntries
CacheCurrentSize
CacheMaximumSize
CacheType

This might also be a good opportunity to remove the safe/unsafe modes since I believe the changes in #22 make these unneeded(afaict).