Skip to content

Doing a better guess for Opossum when tcp/80 is not a/v - #2855

Merged
drwetter merged 1 commit into
3.3devfrom
fix_2847+
Jul 21, 2025
Merged

drwetter merged 1 commit into
3.3devfrom
fix_2847+

Conversation

@drwetter

@drwetter drwetter commented Jul 21, 2025

Copy link
Copy Markdown
Collaborator

This labels the result of a failed test for reaching port 80 when no direct connection is possible as likely not vulnerable.

This seems safe to say, as there's another check whether a proxy is configured, like for corporate environments where a connection is only allowed though the proxy.

What is your pull request about?

  • Bug fix
  • Improvement
  • New feature (adds functionality)
  • Breaking change (bug fix, feature or improvement that would cause existing functionality to not work as expected)
  • Typo fix
  • Documentation update
  • Update of other files

If it's a code change please check the boxes which are applicable

  • For the main program: My edits contain no tabs, indentation is five spaces and any line endings do not contain any blank chars
  • I've read CONTRIBUTING.md and Coding_Convention.md
  • I have tested this fix or improvement against >=2 hosts and I couldn't spot a problem
  • I have tested this new feature against >=2 hosts which show this feature and >=2 host which does not (in order to avoid side effects) . I couldn't spot a problem
  • For the new feature I have made corresponding changes to the documentation and / or to help()
  • If it's a bigger change: I added myself to CREDITS.md (alphabetical order) and the change to CHANGELOG.md

This labels the result of a failed test for reaching
port 80 when no direct connection is possible as likely not
vulnerable.

This seems safe to say, as there's another check whether
a proxy is configured, like for corporate environments where
a connection is only allowed though the proxy.
@drwetter
drwetter merged commit 6c045e0 into 3.3dev Jul 21, 2025
@drwetter
drwetter deleted the fix_2847+ branch July 21, 2025 14:24
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.

1 participant