Skip to content

docs: add SECURITY.md - #431

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

docs: add SECURITY.md#431
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 a security policy covering vulnerability reporting, required report details, response timelines, coordinated disclosure, and security advisory publication.
    • Clarified permitted security testing and directed readers to the vulnerability disclosure statement for additional scope and safe-harbor guidance.

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

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: 06b81d51-aff3-400e-a81d-6d48ea972dc1

📥 Commits

Reviewing files that changed from the base of the PR and between 19c645d and 1bd4ffa.

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

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The repository adds SECURITY.md. The policy defines vulnerability reporting, response timelines, supported-version fixes, testing permissions, disclosure terms, and advisory publication.

Changes

Security Policy

Layer / File(s) Summary
Security policy and reporting process
SECURITY.md
Adds reporting instructions, required vulnerability details, response commitments, supported-version coverage, testing permissions, disclosure references, and advisory publication guidance.

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

Merge Risk: ⚪ Minimal · up to 1bd4f

This adds a repository-local security policy with the intended reporting contact and disclosure-policy link; no actionable merge-blocking risk remains beyond normal review and confirming the planned documentation merge order.

🚥 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 the repository security policy.
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

codacy-production Bot commented Aug 20, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 1 minor

Results:
1 new issue

Category Results
Comprehensibility 1 minor

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 4584222 into main Aug 24, 2026
6 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