Skip to content

Add SECURITY.md - #1312

Closed
jameslamb wants to merge 1 commit into
NVIDIA:mainfrom
jameslamb:docs/security-md
Closed

Add SECURITY.md#1312
jameslamb wants to merge 1 commit into
NVIDIA:mainfrom
jameslamb:docs/security-md

Conversation

@jameslamb

Copy link
Copy Markdown
Member

Description

Contributes to rapidsai/build-planning#281

  • adds a SECURITY.md describing how to report security vulnerabilities

Notes for Reviewers

Why not just set this org-wide?

An org-wide default is set at https://github.com/rapidsai/.github/blob/main/SECURITY.md, but adding an actual file in each repo offers a few benefits:

  • ensures security policy travels with the repo to forks, clones, mirrors, etc.
  • allows per-repo governance over the security policy (via PR review, CODEOWNERS, etc.)

This can be admin-merged

I'll stop CI intentionally after pre-commit runs, to save CI time and resources.

@jameslambjameslamb added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels May 27, 2026
@copy-pr-bot

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

Description

Contributes to rapidsai/build-planning#281

  • adds a SECURITY.md describing how to report security vulnerabilities

Notes for Reviewers

Why not just set this org-wide?

An org-wide default is set at https://github.com/rapidsai/.github/blob/main/SECURITY.md, but adding an actual file in each repo offers a few benefits:

  • ensures security policy travels with the repo to forks, clones, mirrors, etc.
  • allows per-repo governance over the security policy (via PR review, CODEOWNERS, etc.)

This can be admin-merged

I'll stop CI intentionally after pre-commit runs, to save CI time and resources.

I had just added this in a PR #1310 :), but I would like your approach on this as per rapids standards.

@jameslamb

Copy link
Copy Markdown
MemberAuthor

Ha oh great!

Sorry, I hadn't checked all the repos yet, just auto-generated these draft PRs with https://github.com/rapidsai/rapids-reviser

Let's use your PR, though if you don't have strong opinions then I do think you should copy the SECURITY.md from my PR here. We're rolling it out across all of RAPIDS, and its content comes from NVIDIA security folks internally.

ramakrishnap-nv added a commit that referenced this pull request May 28, 2026
PR #1312 (closed) was the canonical "Add SECURITY.md" PR — it shipped
three pieces together and the SECURITY.md on this onboarding-prep PR
only had the file itself. Pulling the two missing infra pieces and
refreshing the content to the PR #1312 wording so the policy doesn't
diverge from the version Ops reviewed.
Changes:
- SECURITY.md: refresh to PR #1312's content. Three reporting channels
(PSIRT web form, psirt@nvidia.com email, GitHub Private Vulnerability
Reporting). Adds the WARNING callout, the Report Details section, and
the PSIRT response-process URL. Fixes the PGP key URL to the canonical
/security/pgp-key path.
- .github/CODEOWNERS: route /SECURITY.md edits to @rapidsai/ops-codeowners
so the disclosure policy isn't edited without Ops review.
- .github/workflows/pr.yaml: add '!SECURITY.md' next to the existing
'!README.md' / '!CONTRIBUTING.md' / '!LICENSE' exclusions in all four
path-filter blocks (cpp-build, python-build, docs-build, conda-build).
Without this, doc-only edits to SECURITY.md trigger the full CI matrix.
Signed-off-by: Ramakrishna Prabhu <ramakrishnap@nvidia.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvementImproves an existing functionalitynon-breakingIntroduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jameslamb@ramakrishnap-nv