Skip to content

docs: [domaintools_support] clarify support status and API coverage - #1072

Open
adulau wants to merge 2 commits into
mainfrom
codex/enrich-domaintools_support
Open

adulau wants to merge 2 commits into
mainfrom
codex/enrich-domaintools_support

Conversation

@adulau

@adulau adulau commented Oct 29, 2026 •

Copy link
Copy Markdown
Member

Document the legacy DomainTools module services and maintenance status, distinguish its API coverage from Iris services, and link the provider-supported MISP integration. Update discovery metadata to point readers to the support guide. This change does not alter request or result handling.

Validated the documented lookups against the existing module and ran flake8 on the metadata change. For maintainer review.

Review updates: Publish the support guide through the existing MkDocs source directory and link discovery metadata to its deployed URL.

@adulau
adulau marked this pull request as ready for review October 29, 2026 22:11
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-29T22:12:41.694592Z 689c904 Draft marked ready
🔒 Security Review ✅ Completed 2026-10-29T22:12:43.146717Z 689c904 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 689c904d85

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +55 to +56
"This legacy module does not appear to be actively maintained. See documentation/expansion/domaintools.md "
"for API coverage and the provider-supported MISP integration.\n\n"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Publish the referenced DomainTools guide

When the documentation site is built through Makefile, prepare_docs copies only documentation/mkdocs/*.md into the MkDocs source directory, so the new documentation/expansion/domaintools.md guide is never deployed. The generated expansion page therefore directs readers to a repository-relative path that is neither a published page nor a Markdown link; copy/include the guide in the MkDocs build and link to its deployed URL.

Useful? React with 👍 / 👎.

This branch has not been deployed

No deployments
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