<p>Hi waja,</p>
<p>You know, i'm not really against this patch, but one weird thing I noticed is that for straight up SSL services, check_http certificate check works! Makes me think the check should actually be implemented in check_tcp for all SSL protocols.</p>
<p>Then once we get there, what prevent us form adding just the required logic in check_tcp to implement the STARTTLS certificate checks for every other STARTTLS-cabaple protocol?</p>
<p>So I don't really mind this merge gets in, but I think the check_tcp suggestion is still worth it and would make this patch obsolete. It's just a mater of which one can get in first I guess...</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/monitoring-plugins/monitoring-plugins/pull/1195#issuecomment-33554590'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/5514713__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNjUwMDczOCwiZGF0YSI6eyJpZCI6MTc2ODU3NzB9fQ==--1fe9b443cbcc52f6e79f9b9438d257d711125002.gif' height='1' width='1'></p>