Skip to content

Fix: add help text to dnsx plugin metadata and refresh checksum#711

Closed
Maria-Niya wants to merge 4 commits into
utksh1:mainfrom
Maria-Niya:update-dnsx-metadata
Closed

Fix: add help text to dnsx plugin metadata and refresh checksum#711
Maria-Niya wants to merge 4 commits into
utksh1:mainfrom
Maria-Niya:update-dnsx-metadata

Conversation

@Maria-Niya

@Maria-Niya Maria-Niya commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Description

Added missing "help" text to the target field in the 'dnsx' metadata (plugins/dnsx/metadata.json). The new copy explains the expected input format and operator intent to the user, preventing confusion when the plugin form is rendered in the UI. Additionally, the plugin checksum is refreshed.

Related Issues

closes #524

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ x ] Documentation update

How Has This Been Tested?

  1. Modified plugins/dnsx/metadata.json locally.
  2. Ran the checksum refresh successfully (python scripts/refresh_plugin_checksum.py --plugin dnsx) and validated with no errors.

Checklist

  • [ x ] My code follows the code style of this project.
  • [ x ] I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • [ x ] I have made corresponding changes to the documentation.
  • [ x ] My changes generate no new warnings.

Copilot AI review requested due to automatic review settings June 9, 2026 04:43

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the dnsx plugin metadata to improve UX and reflect the modified schema/content.

Changes:

  • Switched the plugin icon representation to a literal emoji character.
  • Added a help description for the Domain input field.
  • Updated the plugin checksum to match the modified metadata.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread plugins/dnsx/metadata.json Outdated
"required": true,
"placeholder": "secuscan.in"
"placeholder": "secuscan.in",
"help": "The target domain name to perform DNS resolution and validation against (e.g, example.com)."
@utksh1 utksh1 added level:beginner 20 pts difficulty label for small beginner-friendly PRs type:docs Documentation work category bonus label area:plugins Scanner plugin metadata, schemas, or plugin runtime work gssoc:invalid Admin validation: invalid for GSSoC scoring labels Jun 9, 2026
@utksh1

utksh1 commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Closing this as the lower-grade duplicate of #726. The final diff no longer adds the requested dnsx help text and still fails backend integration/checksum validation. Please continue with a corrected, focused update in #726.

@utksh1 utksh1 closed this Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:plugins Scanner plugin metadata, schemas, or plugin runtime work gssoc:invalid Admin validation: invalid for GSSoC scoring level:beginner 20 pts difficulty label for small beginner-friendly PRs type:docs Documentation work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[PLUGINS] Add field help text for dnsx metadata

3 participants