Skip to content

docs: add SECURITY.md - #274

Merged
dpage merged 2 commits into
mainfrom
docs/security-policy
Aug 24, 2026
Merged

docs: add SECURITY.md#274
dpage merged 2 commits into
mainfrom
docs/security-policy

Conversation

@AntTheLimey

@AntTheLimey AntTheLimey commented Aug 20, 2026

Copy link
Copy Markdown
Member

Adds SECURITY.md to the repository root. It names security@pgedge.com as
the single reporting route and points at the pgEdge Vulnerability Disclosure
Statement for scope, safe harbour and CVE handling.

The file is identical in every pgEdge product repository — nothing in it is
repo-specific.

Why an in-repo copy when there is an org default

pgEdge/.github carries the same file as an organisation default, which covers
every repository that has none of its own. Defaults do not appear in a
repository's file tree, git history, clones or release archives — only in the
Security tab. A product a customer clones or vendors should carry its own
policy, and OpenSSF Scorecard's security-policy check only looks in the
repository itself.

Draft on purpose — merge order matters

The only link in this file is https://docs.pgedge.com/security, and that URL
returns 404 today. Merging before the statement is live publishes a
security policy whose one actionable link is dead.

Merge order:

  1. docs: add vulnerability disclosure statement pgedge-docs#138 — publishes docs.pgedge.com/security. Out of
    draft and awaiting review.
  2. docs: add org-wide SECURITY.md default .github#6 — the org-wide default.
  3. This PR, alongside the other product repositories.

No action needed from you until #138 merges. Reviews welcome now.

Summary by CodeRabbit

  • Documentation
    • Added security guidance covering vulnerability reporting, supported versions, authorized testing, safe-harbour terms, disclosure practices, and published advisories.

Points at security@pgedge.com as the single reporting route and at the
pgEdge Vulnerability Disclosure Statement for scope, safe harbour and CVE
handling. Identical across every pgEdge product repository.

Do not merge before pgEdge/pgedge-docs#138 publishes
docs.pgedge.com/security; until it does, the only link in this file 404s.
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: c2516f39-7d47-4fdb-b40b-5d7360c74f4a

📥 Commits

Reviewing files that changed from the base of the PR and between b971c41 and f61b09d.

📒 Files selected for processing (1)
  • SECURITY.md

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


Walkthrough

Added SECURITY.md with vulnerability reporting instructions, supported-version coverage, disclosure guidance, authorized testing terms, and repository advisory information.

Changes

Security Policy

Layer / File(s) Summary
Security policy guidance
SECURITY.md
Documents private vulnerability reporting via security@pgedge.com, required report details, five-business-day acknowledgement, latest-release support, disclosure and CVE guidance, authorized testing terms, and repository security advisories.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to f61b0

This PR adds a repository security policy without introducing a concrete correctness, security, or availability issue. No actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the addition of SECURITY.md, which is the main change in the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/security-policy

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

The statement URL and the reporting address were bare text. Dave asked
for the URL to be a real link on pgedge-safesession#73: GitHub
autolinks it, but nothing guarantees another viewer will, and the two
actionable things in a security policy should not depend on a
renderer. The published statement at docs.pgedge.com/security already
writes the address as an explicit mailto link, so this keeps the two
documents consistent.

Identical across every repo carrying this file.
@AntTheLimey
AntTheLimey marked this pull request as ready for review August 24, 2026 13:46
@AntTheLimey
AntTheLimey requested a review from dpage August 24, 2026 13:56
@dpage
dpage merged commit 1a50f4c into main Aug 24, 2026
12 checks passed
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.

2 participants