Skip to content

Fix DNSdumpster CSRF extraction and add tests/CI - #413

Open
mazennafee wants to merge 8 commits into
aboul3la:masterfrom
mazennafee:fix/dnsdumpster-csrf-and-subbrute-ns
Open

Fix DNSdumpster CSRF extraction and add tests/CI#413
mazennafee wants to merge 8 commits into
aboul3la:masterfrom
mazennafee:fix/dnsdumpster-csrf-and-subbrute-ns

Conversation

@mazennafee

Copy link
Copy Markdown

Summary

This PR fixes a crash when DNSdumpster omits its CSRF token by:

  • Implementing a defensive get_csrftoken (regex + BeautifulSoup fallback).
  • Delegating class method to the module-level function.
  • Adding a unit test reproducing the missing-token case.
  • Adding requirements.txt, LICENSE, and a GitHub Actions CI workflow.

Testing

  1. python -m venv .venv && source .venv/bin/activate
  2. pip install -r requirements.txt
  3. pip install -e .
  4. python -m pytest -q

Notes

Backwards compatible when DNSdumpster provides a token. Happy to split docs/CI/tests into separate PRs if preferred.
PR

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.

1 participant