Skip to content

Fix IPv6 addresses#2530

Merged
drwetter merged 1 commit into3.2from
fix_ipv6_ula_and_more
Jul 22, 2024
Merged

Fix IPv6 addresses#2530
drwetter merged 1 commit into3.2from
fix_ipv6_ula_and_more

Conversation

@drwetter
Copy link
Copy Markdown
Collaborator

@drwetter drwetter commented Jul 22, 2024

Local and ULA and more IPv6 adresses were incorrectly filtered by awk '/^[0-9]/ { print $1 }' which searches in the first term for numeric values only.

This PR adds a-f and fixes #2529 .

Describe your changes

Please refer to an issue here or describe the change thoroughly in your PR.

What is your pull request about?

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature 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 and the indentation is five spaces
  • I've read CONTRIBUTING.md and Coding_Convention.md
  • I have tested this fix 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

Local and ULA and more IPv6 adresses were incorrectly filtered by ``awk '/^[0-9]/ { print $1 }'``
which searches in the first term for only numeric values.

The PR adds a-f and fixes #2529 .
@drwetter drwetter merged commit fa9fe27 into 3.2 Jul 22, 2024
@drwetter drwetter deleted the fix_ipv6_ula_and_more branch July 22, 2024 21:31
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.

testssl 3.2: Are local addresses deliberately filtered?

1 participant