Skip to content

Conversation

@colin128184-lgtm
Copy link

Added a security policy document outlining supported versions and vulnerability reporting.

Added a security policy document outlining supported versions and vulnerability reporting.
@colin128184-lgtm colin128184-lgtm requested a review from a team as a code owner December 3, 2025 09:54
Copy link

@codacy-production codacy-production bot left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

SECURITY.md added. No blocking code or security issues detected by Codacy. Before merging, the policy should include concrete reporting instructions and clearer supported-versions semantics; these are important for an effective security process but not merge-blocking.

About this PR

  • Add tests/coverage are not applicable for this change (documentation only). If your project requires CI checks for documentation, ensure SECURITY.md passes any linting/markdown checks.
Low confidence findings
  • PR description is minimal — consider summarizing the policy contents and intended audience (maintainers vs public), and whether this is a repository-level or organization-wide policy.

💡 Codacy uses AI. Check for mistakes.

Comment on lines +15 to +21
## Reporting a Vulnerability

Use this section to tell people how to report a vulnerability.

Tell them where to go, how often they can expect to get an update on a
reported vulnerability, what to expect if the vulnerability is accepted or
declined, etc.

Choose a reason for hiding this comment

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

❗ Issue

The "Reporting a Vulnerability" section is generic. Add concrete instructions: an email address (preferably security@yourdomain), expected acknowledgment timeframe (e.g., 48 hours), disclosure policy (coordinated disclosure timeline), and a PGP key or link for secure reports if you accept them. Include whether reporters can expect bounty information or triage steps.

🟡 Medium risk

Comment on lines +8 to +13
| Version | Supported |
| ------- | ------------------ |
| 5.1.x | :white_check_mark: |
| 5.0.x | :x: |
| 4.0.x | :white_check_mark: |
| < 4.0 | :x: |

Choose a reason for hiding this comment

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

💡 Suggestion

The supported versions table uses emojis (:white_check_mark:, :x:) which may not render consistently in all clients or when consumed programmatically. Prefer explicit text (e.g., "Yes" / "No" or "Supported" / "Not supported") and define what "supported" means (security fixes only, bug fixes, timeframe). Example replacement:

This might be a simple fix:

Suggested change
| Version | Supported |
| ------- | ------------------ |
| 5.1.x | :white_check_mark: |
| 5.0.x | :x: |
| 4.0.x | :white_check_mark: |
| < 4.0 | :x: |
Version | Supported |
------- | --------- |
5.1.x | Supported (security updates)
5.0.x | Not supported
4.0.x | Supported (security updates)
< 4.0 | Not supported

🟡 Medium risk

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