Skip to content

Add --abort-no-tls switch#2279

Closed
ikstream wants to merge 1 commit intotestssl:3.1devfrom
ikstream:add-abort-no-tls-flag
Closed

Add --abort-no-tls switch#2279
ikstream wants to merge 1 commit intotestssl:3.1devfrom
ikstream:add-abort-no-tls-flag

Conversation

@ikstream
Copy link
Copy Markdown

When used in automated pipelines, processing output from other tools with testssl might cause a halt of the whole pipeline if the previous output is detected false positive as TLS/SSL enabled Server.

Enabling this switch will cause testssl to return ALLOK after the first sign, that TLS/SSL is not enabled on the server.

When used in automated pipelines, processing output from other tools
with testssl might cause a halt of the whole pipeline if the previous
output is detected false positive as TLS/SSL enabled Server.

Enabling this switch will cause testssl to return ALLOK after the first
sign, that TLS/SSL is not enabled on the server.
@dcooper16
Copy link
Copy Markdown
Collaborator

Rather than defining a new flag for this one particular scenario, why not just use --warnings batch?

@ikstream
Copy link
Copy Markdown
Author

Thank you for pointing this out. I have been looking for an option like this, but haven't found one.

@ikstream ikstream closed this Nov 24, 2022
dcooper16 pushed a commit to dcooper16/testssl.sh that referenced this pull request Nov 25, 2022
This commit fixes incorrect indentations introduced in testssl#2276 and testssl#2278, and also removes the extra whitespace that would have been removed by testssl#2279.
@dcooper16 dcooper16 mentioned this pull request Nov 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants