Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Change regexp to match both ipv4 and ipv6 addresses #65

Closed
wants to merge 2 commits into from
Closed

Change regexp to match both ipv4 and ipv6 addresses #65

wants to merge 2 commits into from

Conversation

iamargus95
Copy link

@iamargus95 iamargus95 commented Apr 10, 2023 •
edited

  • Update regex to capture ipv6 ranges as well.
  • WIP: Add test case for IPv6 ranges.
  • Added fastly IPv6 range to sources_data.json to facilitate tests.

@iamargus95
Change regex to match both ipv4 and ipv6 ranges
c417272 
Signed-off-by: iamargus95 <kamathsuraj95@gmail.com>
@iamargus95 iamargus95 marked this pull request as draft April 10, 2023 04:24
@iamargus95 iamargus95 mentioned this pull request Apr 10, 2023
Open
@ShubhamRasal ShubhamRasal linked an issue Apr 10, 2023 that may be closed by this pull request
Add support for ipv6 ranges #59
Open
@iamargus95
WIP: Add test case for IPv6 regex extraction
517f302 
Signed-off-by: iamargus95 <kamathsuraj95@gmail.com>
@sonarcloud
Copy link

sonarcloud bot commented Apr 10, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 1 Security Hotspot
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ehsandeep ehsandeep deleted the branch projectdiscovery:dev April 25, 2023 21:30
@ehsandeep ehsandeep closed this Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@iamargus95 iamargus95

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add support for ipv6 ranges
2 participants
@iamargus95 @ehsandeep