Skip to content

feat: add redirect-chain blocklist check, tune urlscan polling - #49

Merged
calebephrem merged 1 commit into
open-devhub:mainfrom
calebephrem:main
Jul 21, 2026
Merged

calebephrem merged 1 commit into
open-devhub:mainfrom
calebephrem:main

Conversation

@calebephrem

Copy link
Copy Markdown
Member

What does this PR do?

  • scan command now resolves a URL's full redirect chain and checks each hop against a local blocklist of known IP logger/grabber domains before running external scans (#44
  • also scan command's urlscan.io polling now runs 5 attempts at 3s intervals instead of a 10s wait plus 8x5s polls (#44)

Closes

List any issues that are linked to this pull request (e.g., Closes #123).

Type of change(s)

  • Bug fix
  • New feature
  • Documentation update
  • Style / UX change
  • Refactor (no functional change)
  • Performance improvement
  • Other

Checklist

  • I've tested this change locally and it works as expected
  • bun run typecheck or npm run typecheck completes without errors
  • My commit messages follow Conventional Commits (e.g. fix: ..., feat: ..., chore: ...)
  • I've updated relevant docs (README, comments, etc.) if this change needs it
  • No leftover console.log or debug code

Screenshots / recordings (if applicable)

image (bitly is url shortener platform, i did this to make sure it scans all the redirects of a url)

@devhub-bot devhub-bot Bot added the feat New feature label Jul 21, 2026
@devhub-bot

devhub-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Note

Linting checks passed successfully 🎉

All formatting and code quality checks are clean.

You're good to merge 🚀

@calebephrem
calebephrem merged commit 5235502 into open-devhub:main Jul 21, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue with ;scan command

1 participant