Repository navigation
docs: make the security policy org-wide - #48
Merged
Merged
Conversation
Add generic forms of the additions of foca-plugins' SECURITY.md to the org security policy: reports name the affected packages, the versions of the components they run with and the configuration; repositories that publish several packages release fixes for each affected package and any package that pins it; vulnerabilities go to the repository of the component that has them. State that repositories have no SECURITY.md of their own, and replace the deprecated name "ELIXIR Cloud & AAI" with "ELIXIR On-Cloud". Closes #40 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates SECURITY.md to serve as the organisation-wide ELIXIR On-Cloud security policy, clarifying report routing, required vulnerability context, and package-level remediation expectations. File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
uniqueg
marked this pull request as ready for review
November 1, 2026 03:28
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="SECURITY.md" line_range="4-5" />
<code_context>
This policy applies to every repository of
-[ELIXIR Cloud & AAI](https://github.com/elixir-cloud-aai) that has no
-`SECURITY.md` of its own; repositories with their own policy follow that one.
+[ELIXIR On-Cloud](https://github.com/elixir-cloud-aai); repositories don't
+have their own `SECURITY.md`.
## Reporting a vulnerability
</code_context>
<issue_to_address>
**Repository reporters see old guidance**
When an organization repository retains its own `SECURITY.md`, including foca-plugins, gitHub serves that repository’s `SECURITY.md` instead of this organization default, so its reporters see different guidance and this policy does not apply to them.
Remove or update repository-local `SECURITY.md` files so reporters receive the intended policy.
</issue_to_address>Linked issues
#40 docs: make the security policy org-wide · 5 of 5 met
| # | Requirement | Status | Notes |
|---|---|---|---|
| 1 | Reports name the affected repository and package(s), their versions, component versions, and relevant configuration without secrets. | ✅ Met | The reporting checklist asks for the affected repository and package versions, component versions, and relevant configuration without secrets. |
| 2 | For repositories publishing several packages, release fixes for each affected package and any package that pins it. | ✅ Met | The supported-versions section includes this guidance. |
| 3 | Report vulnerabilities to the repository of the affected component; if unsure, report where found so maintainers can route it. | ✅ Met | The reporting section gives this direction and the FOCA example. |
| 4 | Replace the statement that repositories with their own policy follow that policy with a statement that repositories do not have their own SECURITY.md. | ✅ Met | The opening now states that repositories don't have their own SECURITY.md. |
| 5 | Replace “ELIXIR Cloud & AAI” with “ELIXIR On-Cloud”. | ✅ Met | The policy names ELIXIR On-Cloud. |
Sourcery assessment
Approval pending. 1 finding to address first.
Blocking findings: SECURITY.md:5
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Makes
SECURITY.mdthe security policy of every repository of theorganisation, so that foca-plugins' own
SECURITY.mdcan go (#38), by addinggeneric forms of its additions:
the versions of the components they run with (e.g., the FOCA version of an
app) and the relevant configuration (without secrets);
affected package, and for any package that pins it;
FOCA for the framework); if unsure, report where you found it, and the
maintainers route it;
repositories don't have their own
SECURITY.md;Closes #40
Light lane: documentation text only (no code, workflows or configuration),
so no reviewer subagents; I reviewed the diff myself.
Verification
pre-commit run --all-filesandpre-commit run --all-files --hook-stage manual gitleaks-all: all hookspassed (markdownlint included).
Declined review findings
own
SECURITY.mdstill override this one). Intended: removingfoca-plugins'
SECURITY.mdis foca-plugins#35, which this issue blocks.Open questions
None.
🤖 Generated with Claude Code