这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@deehyeon
Copy link

Summary

This PR adds a new passive source integration for urlscan.io into Subfinder.

Changes

  • Added new source file: pkg/subscraping/sources/urlscan/urlscan.go
  • Implemented the Source interface with pagination, deduplication, and basic retry/backoff
  • Registered urlscan in AllSources (passive.go)
  • Updated tests in sources_test.go to include urlscan

Motivation

urlscan.io provides subdomain data from scanned URLs and pages.
Integrating it into Subfinder increases overall passive coverage and improves discovery results.

Related Issue

Closes #1643

Copy link
Member

@ehsandeep ehsandeep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deehyeon thanks for adding the new source!

It looks like you've committed a binary build in your PR, could you please remove it?

@deehyeon
Copy link
Author

@ehsandeep Thanks for pointing that out! 🙏
I’ve removed the committed binary from the PR.

@deehyeon
Copy link
Author

@ehsandeep I re-ran the tests locally with go test ./pkg/passive -run TestSourcesWithoutKeys -v
and all cases (rapiddns, threatcrowd, etc.) passed successfully.

The Ubuntu CI failure seems unrelated to the urlscan implementation —
it's due to flaky external services (rapiddns, threatcrowd) returning
timeouts or invalid responses at that time.

The urlscan source itself is working correctly.

Screenshot 2025-09-22 at 11 08 44 AM

@deehyeon deehyeon requested a review from ehsandeep September 22, 2025 12:01
@dogancanbakir
Copy link
Member

Just a reminder: #957 (comment)

@deehyeon
Copy link
Author

@dogancanbakir Thanks for the review. I understand the concern about strict rate limits—it makes sense. Would you like me to close this PR, or will you handle it? I’m fine either way.

@dogancanbakir dogancanbakir mentioned this pull request Oct 1, 2025
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.

Add urlscan.io

3 participants