aboutsummaryrefslogtreecommitdiffstats
path: root/net-mgmt/xymon-server/files/patch-fix_tls13_banner
blob: ebcffdcfdf3041e616d316b459b383283550021b (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
From: Andreas Oberritter <obi@saftware.de>
Date: Tue, 11 May 2021 13:05:10 +0200
Subject: Fix reading service banners from TLS-1.3-enabled hosts

Bug-Debian: https://bugs.debian.org/930532
Forwarded: https://lists.xymon.com/archive/2024-February/048290.html
Last-Update: 2021-05-11


Related discussion:
https://lists.xymon.com/archive/2019-July/046585.html
https://lists.xymon.com/archive/2019-November/046893.html
https://lists.xymon.com/archive/2020-February/046986.html
https://lists.xymon.com/archive/2020-March/046987.html
https://sourceforge.net/p/xymon/discussion/435278/thread/c0359f08e7/
---
 xymonnet/contest.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

--- xymonnet/contest.c
+++ xymonnet/contest.c
@@ -1347,13 +1347,12 @@ restartselect:
 							}
 						}
 
-						if ((item->svcinfo->flags & TCP_HTTP) && 
-						    ((res > 0) || item->sslagain)     &&
-						    (!datadone) ) {
+						if (((item->svcinfo->flags & TCP_HTTP) && res > 0) || item->sslagain) {
 							/*
-							 * HTTP : Grab the entire response.
+							 * Grab the entire HTTP response or wait for
+							 * TLS handshake to complete.
 							 */
-							wantmoredata = 1;
+							wantmoredata = !datadone;
 						}
 
 						if (!wantmoredata) {