Skip to content

Add security policy#1090

Merged
jamisonbryant merged 2 commits into
5.xfrom
add-security-policy
May 28, 2026
Merged

Add security policy#1090
jamisonbryant merged 2 commits into
5.xfrom
add-security-policy

Conversation

@jamisonbryant
Copy link
Copy Markdown
Contributor

@jamisonbryant jamisonbryant commented May 28, 2026

Summary

Add a security policy to document supported versions and the vulnerability reporting process.

Major Changes

  • Added .github/SECURITY.md with supported version table reference to version table in wiki and vulnerability reporting instructions

Minor Changes

None.

Backwards Compatibility Notes

No breaking changes. This is a documentation-only addition.

Work Remaining

None.

Test Plan

N/A - documentation only.

Comment thread .github/SECURITY.md Outdated

| Version | Supported | Security fixes until
| ------- | ------------------ | --------------------
| 5.x | :white_check_mark: | Currently supported
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I strongly advise against using a concrete table here, just mention the active versions in version map, wherever it is
https://github.com/cakephp/migrations/wiki#version-map
And base this info on that, we should be not as strict (e.g. date must not be included either), otherwise this is maintenance hell.

The EOL versions will have [EOL], so those are clearly not part of it anymore.

Otherwise we will have outdated and wrong docs somewhere due to the redundancy.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoids duplicating version support info that is already
maintained in the wiki, reducing maintenance burden.
@jamisonbryant jamisonbryant merged commit a5f2d90 into 5.x May 28, 2026
20 checks passed
@jamisonbryant jamisonbryant deleted the add-security-policy branch May 28, 2026 15:32
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