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.

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

Add a Integration Test to scan the top 1k domains and validate that ZDNS's result is correct #370

Merged
merged 25 commits into from
May 29, 2024
Merged
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
92412fd
testing if github runner can scan 1k domains without crazy timeouts
phillip-stephens May 24, 2024
bb83377
seeing results
phillip-stephens May 24, 2024
75e5fdc
able to request X domain to Y IP
phillip-stephens May 24, 2024
ab3c87c
working a and alookup tests
phillip-stephens May 24, 2024
2a789ce
working validators
phillip-stephens May 28, 2024
dec7f03
cleaning up unneeded files and added some logs
phillip-stephens May 28, 2024
f48992e
cleanup and add as github action
phillip-stephens May 28, 2024
202d766
updated zdns exe path
phillip-stephens May 28, 2024
f706612
updated test name in ci.yml
phillip-stephens May 28, 2024
48a16c6
made python test a script
phillip-stephens May 28, 2024
2eeeae9
updated script permissions
phillip-stephens May 28, 2024
1855325
removed dead import
phillip-stephens May 28, 2024
f5cc9e3
updated file name
phillip-stephens May 28, 2024
e0ca48b
made error msg less scary
phillip-stephens May 28, 2024
fb07c87
only run CI on commits to main branch and all PR's, otherwise we're d…
phillip-stephens May 28, 2024
b7005b1
remove limit to top 100, run against all 1k domains
phillip-stephens May 28, 2024
e9e7ee2
moved to only scanning 100 domains
phillip-stephens May 28, 2024
581fa70
cleanup
phillip-stephens May 28, 2024
4b04b9f
fixed name of CI action
phillip-stephens May 28, 2024
fc889fd
added rest of domains
phillip-stephens May 28, 2024
1b87244
moved to 500 domains
phillip-stephens May 28, 2024
ac820d1
moved back to 100 domains
phillip-stephens May 28, 2024
6399896
Merge branch 'main' into phillip/large-scale-integration
phillip-stephens May 28, 2024
bed56ec
add comments explaining integration tests
phillip-stephens May 28, 2024
cdb6659
Merge branch 'phillip/large-scale-integration' of github.com:zmap/zdn…
phillip-stephens May 28, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
made error msg less scary
  • Loading branch information
phillip-stephens committed May 28, 2024
commit e0ca48b08e1cbe994696596d6cac2c5ea42346a7
4 changes: 2 additions & 2 deletions testing/top_1k_domains_integration_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ def get_zdns_results_a(domains: List[str]) -> List[tuple[str, str]]:
ip = line.split('"answer":"')[1].split('"')[0]
domain_ip_pairs.append((domain, ip))
else:
print(f"Failed to resolve domain: {line}")
print(f"Failed to parse ZDNS output: {line}")
return domain_ip_pairs


Expand All @@ -134,7 +134,7 @@ def get_zdns_results_a_lookup(domains: List[str]) -> List[tuple[str, str]]:
domain = line_data.get('name')
domain_ip_pairs.append((domain, ip))
else:
print(f"Failed to resolve domain: {line}")
print(f"Failed to parse ZDNS output: {line}")
return domain_ip_pairs


Expand Down