-
-
Notifications
You must be signed in to change notification settings - Fork 166
docs: add security policy #2127
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 3.x
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,35 @@ | ||||||
| # Tempest security policy | ||||||
|
|
||||||
| ## Reporting a security issue | ||||||
|
|
||||||
| If you think you have found a security issue within Tempest, don't create a GitHub issue and don't publish a pull request with proof of concept. In the first instance, report issues using [GitHub's security advisory reporting mechanism](https://github.com/tempestphp/tempest-framework/security/advisories/new), with as much information as you can provide, ideally including steps-to-recreate. Security reports submitted on this page are forwarded to the core maintainers only. | ||||||
|
|
||||||
| The core maintainers will determine whether this is classified as a security issue, and address it accordingly, or whether it is classified as a regular bug, and may ask you to raise a GitHub issue instead, at this time. | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
|
||||||
| ## Resolution process | ||||||
|
|
||||||
| The core maintainers will aim to acknowledge and validate any reported security issue promptly. | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
|
||||||
| Following the validation of a security issue, the core maintainers will broadly: | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
|
||||||
| 1. Work on a patch and commit it to the repository via GitHub following the usual processes. | ||||||
|
|
||||||
| 2. Issue a release containing the security release. | ||||||
|
|
||||||
| 3. Consider offering a Rector automated fix within the release, where appropriate. | ||||||
|
|
||||||
| 4. Notify all subscribed Tempest parties via the usual channels (discord, blog, etc) that the updated is published. | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
|
||||||
| ## Keeping Tempest secure | ||||||
|
|
||||||
| Several controls are in place to ensure that Tempest code releases are kept secure. | ||||||
|
|
||||||
| 1. All maintainers with write access to the repository use multi-factor authentication. | ||||||
|
|
||||||
| 2. Branch protection is configured on the repository. | ||||||
|
|
||||||
| 3. All access rights and privileges (including automated accounts, API keys) are assigned on a Principle of Least Privilege basis. | ||||||
|
|
||||||
| 4. Every pull request requires the successful completion of code quality and static analysis checks, and is reviewed by a core maintainer. | ||||||
|
|
||||||
| 5. Tempest actively upgrades dependencies based on deprecations and notices from upstream packages where used. | ||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.