Skip to content

Commit a28f562

Browse files
committed
Merge branch 'dev' of github.com:nabla-c0d3/sslyze into dev
2 parents bea0f8c + 1eb0feb commit a28f562

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

sslyze/server_connectivity.py

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -388,6 +388,17 @@ def _detect_client_auth_requirement_with_tls_1_3(
388388
else:
389389
raise
390390

391+
except OSError as e:
392+
# If these errors get propagated here, it means they're not part of the known/normal errors that
393+
# can happen when trying to check requirements for client auth and defined in tls_connection.py
394+
# Hence we re-raise these as "unknown" connection errors; might be caused by bad connectivity to
395+
# the server (random disconnects, etc.) and the scan against this server should not be performed
396+
raise ConnectionToServerFailed(
397+
server_location=server_location,
398+
network_configuration=network_config,
399+
error_message=f'Unexpected connection error: "{e.args}"',
400+
)
401+
391402
finally:
392403
ssl_connection_auth.close()
393404

0 commit comments

Comments
 (0)