Skip to content

docs(security): add initial security policy#1721

Merged
tmccombs merged 1 commit intosharkdp:masterfrom
janderssonse:fix/add-security-policy
May 20, 2025
Merged

docs(security): add initial security policy#1721
tmccombs merged 1 commit intosharkdp:masterfrom
janderssonse:fix/add-security-policy

Conversation

@janderssonse
Copy link
Copy Markdown
Contributor

@janderssonse janderssonse commented May 11, 2025

This PR adds a SECURITY.md file, battle tested in other projects and orgs, (the construct is CC0 ie public domain, for example from here https://raw.githubusercontent.com/itiquette/git-provider-sync/refs/heads/main/SECURITY.md so just reuse)

It is identical to this ! sharkdp/bat#3290 except for the example url.

A SECURITY.md would help anyone assessing the project for use, give a hint of how it handles critical no public security issues, and give anyone a clear instruction on how to report them non public.

IE, for someone thinking about using fd in an organization or privately it would give an extra trust factor.

This policy basically says "send your findings, and we will see if we handle them, we will notify you".

Besides, being a good FOSS practice, makes the project look more professional and it is heavily supported by GitHub https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/creating-a-default-community-health-file etc as one of the community health files, so it will pop up automatically in the UI for the end user.

Examples:
Security Tab in project front will be added automatically
Skärmbild från 2025-05-11 05-57-27

Security Policy in the top right corner of UI will be added automatically

Skärmbild från 2025-05-11 05-58-05

Security Policy under Security Overview for the project will have the Security Policy green and enabled.
Skärmbild från 2025-05-11 05-58-23

NOTE: there is a <...> in the text, where the preferred channel for reporting should be added I left that for you, (or tell me what to add there, and I'll rebase with that).

NOTE: I had this in multiple orgs and projects over the years. Only once I had a report, so I don't think one should be worry about getting to much reports from this, this is at least my experience.

Signed-off-by: Josef Andersson <janderssonse@proton.me>
@janderssonse
Copy link
Copy Markdown
Contributor Author

janderssonse commented May 11, 2025

Same as in the Bat PR -> It fails without a changelog entry, but I think this applies (I see this as docs). >[!NOTE]

For PRs, a CI workflow verifies that a suitable changelog entry is
added. If such an entry is missing, the workflow will fail. If your
changes do not need an entry to the changelog (see above), that
workflow failure can be disregarded.

@tmccombs tmccombs merged commit 8b91260 into sharkdp:master May 20, 2025
18 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