This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 618fbae2c6 Improve readability and content of the security policy
(#36023)
618fbae2c6 is described below
commit 618fbae2c615b998f400dc14ddc3089ce614559e
Author: Jarek Potiuk <[email protected]>
AuthorDate: Sun Dec 3 11:22:49 2023 +0100
Improve readability and content of the security policy (#36023)
Our security policy was quite a bit too long and to unstructured
to provide good guideline for the Security Reporters. It was basically
a wall of text where - even if you wanted - you could not easily find
the part that you could be really interested in - for example assessing
the severity of the issue or what happens after you reported the issue.
The change is also not about the structure but also about the content
that resulted from a number of discussions we had with securtity
reporters - answering straight away the questions they had - for
example how long they should wait for answer, how should they
monitor if the CVE has been announced etc. etc.
Having a structured form also allows to link to the particular
sections of the policy in case someone had not read and follow it -
having it structured and linkable, make it much more fair to a) expect
they will be read, b) point out (including pointing out negligence
by the reporter) to the particular section. That sounds fair and
reasonable if made it clear and directly answering questions the
reporters might have.
---
.github/SECURITY.md | 101 ++++++++++++++++++++++++++++++++++--------------
.pre-commit-config.yaml | 2 +-
2 files changed, 74 insertions(+), 29 deletions(-)
diff --git a/.github/SECURITY.md b/.github/SECURITY.md
index 64bd990cbc..6b526c3a6e 100644
--- a/.github/SECURITY.md
+++ b/.github/SECURITY.md
@@ -17,13 +17,24 @@
under the License.
-->
+<!-- START doctoc generated TOC please keep comment here to allow auto update
-->
+<!-- DON'T EDIT THIS SECTION, INSTEAD RE-RUN doctoc TO UPDATE -->
+
+- [What should be and should NOT be reported
?](#what-should-be-and-should-not-be-reported-)
+- [How to report the issue ?](#how-to-report-the-issue-)
+- [Is this really a security vulnerability
?](#is-this-really-a-security-vulnerability-)
+- [How do we assess severity of the issue
?](#how-do-we-assess-severity-of-the-issue-)
+- [What happens after you report the issue
?](#what-happens-after-you-report-the-issue-)
+- [Does CVE in Airflow Providers impact Airflow core package
?](#does-cve-in-airflow-providers-impact-airflow-core-package-)
+- [Where do I find more information about Airflow Security
?](#where-do-i-find-more-information-about-airflow-security-)
+
+<!-- END doctoc generated TOC please keep comment here to allow auto update -->
+
This document contains information on how to report security vulnerabilities
in Apache Airflow and
how security issues reported to the Apache Airflow security team are handled.
If you would like
to learn more, head to the
[Airflow
security](https://airflow.apache.org/docs/apache-airflow/stable/security/)
documentation.
-## Reporting Vulnerabilities
-
**⚠️ Please do not file GitHub issues for security vulnerabilities as they are
public! ⚠️**
The Apache Software Foundation takes security issues very seriously. Apache
@@ -32,56 +43,90 @@ around its features. If you have any concern around Airflow
Security or believe
you have uncovered a vulnerability, we suggest that you get in touch via the
e-mail address
[[email protected]](mailto:[email protected]).
+Before sending the report, however, please read the following guidelines
first. The guidelines should
+answer the most common questions you might have about reporting
vulnerabilities.
+
+### What should be and should NOT be reported ?
+
**Only** use the security e-mail address to report undisclosed security
vulnerabilities in Apache
Airflow and to manage the process of fixing such vulnerabilities. We do not
accept regular
bug reports or other security-related queries at this address. We will ignore
mail
sent to this address that does not relate to an undisclosed security problem
in the Apache Airflow project. Please follow regular communication channels
described in
-the [Airflow Community](https://airflow.apache.org/community/index.html) page
for
-inquiries, questions and other discussions related to the process or issues.
+the [Airflow Community](https://airflow.apache.org/community/index.html) page
for inquiries, questions and other discussions related
+to the process or issues.
+
+Specifically, we will ignore results of security scans that contain a list of
dependencies of Airflow
+with dependencies in Airflow Docker reference image - there is a page that
describes how the
+[Airflow reference Image is fixed at release
time](https://airflow.apache.org/docs/docker-stack/index.html#fixing-images-at-release-time)
and providing helpful instructions explaining
+how you can build your own image and manage dependencies of Airflow in your
own image.
-Specifically, we will ignore results of security scans that contain a list of
-dependencies of Airflow with dependencies in Airflow Docker reference image -
there
-is a page that describes how the
-[Airflow reference Image is fixed at release
time](https://airflow.apache.org/docs/docker-stack/index.html#fixing-images-at-release-time)
-and providing helpful instructions explaining how you can build your
-own image and manage dependencies of Airflow in your own image.
+### How to report the issue ?
Please send one plain-text email for each vulnerability you are reporting
including an explanation
of how it affects Airflow security. We may ask that you resubmit your report
if you send it as an image,
movie, HTML, or PDF attachment when you could as easily describe it with plain
text.
-Before reporting vulnerabilities, please make sure to read and understand the
-[security
model](https://airflow.apache.org/docs/apache-airflow/stable/security/security_model.html)
of Airflow, because
-some of the potential security vulnerabilities that are valid for projects
that are publicly accessible
-from the Internet, are not valid for Airflow.
+### Is this really a security vulnerability ?
+Before reporting vulnerabilities, please make sure to read and understand the
[security
model](https://airflow.apache.org/docs/apache-airflow/stable/security/security_model.html)
+of Airflow, because some of the potential security vulnerabilities that are
valid for projects that are
+publicly accessible from the Internet, are not valid for Airflow.
Airflow is not designed to be used by untrusted users, and some trusted users
are trusted enough to do a
variety of operations that could be considered as vulnerabilities in other
products/circumstances.
-Therefore, some potential security vulnerabilities do not apply to Airflow, or
have a different severity
-than some generic scoring systems (for example `CVSS`) calculation suggests.
Severity of the issue is
-determined based on the criteria described in the
-[Severity Rating blog post](https://security.apache.org/blog/severityrating/)
by the Apache Software
-Foundation Security team.
-The [Airflow Security
Team](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#security-team)
will get back to you after assessing the report.
-If you wish to know more about the ASF security process, the
-[ASF Security team's page](https://www.apache.org/security/) describes
-how vulnerability reports are handled in general by all ASF projects, and
includes PGP keys if
-you wish to use them when you report the issues.
+Again. Please make sure to read and understand the [security
model](https://airflow.apache.org/docs/apache-airflow/stable/security/security_model.html)
+of Airflow - this might help to save your time for reporting and our time for
assessing the issues when
+they are clearly expected, according to the Security model. You also avoid
disappointment when you spend
+a lot of time on preparing the issue report to follow the guidelines above and
what you get as a response is
+"this is not a security issue" or "this is invalid security issue" or "this is
expected behaviour". You
+save time for yourself and for the Airflow Security team by reading and
understanding the security model
+before reporting the issue.
-## Security vulnerabilities in Airflow and Airflow community managed providers
+### How do we assess severity of the issue ?
+
+Severity of the issue is determined based on the criteria described in
+the [Severity Rating blog
post](https://security.apache.org/blog/severityrating/) by the Apache Software
Foundation Security team.
+
+Due to reasons explained in the previous chapter, some potential security
vulnerabilities
+do not apply to Airflow, or have a different severity than some generic
scoring systems
+(for example `CVSS`) calculation suggests. So we are not using any generic
scoring system.
+
+
+### What happens after you report the issue ?
+
+The [Airflow Security
Team](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#security-team)
will get back to you after assessing the report. You will usually get
+confirmation that the issue is being worked (or that we quickly assessed it as
invalid) within several
+business days. Note that this is an Open-Source projects and members of the
security team are volunteers
+so please make sure to be patient. If you do not get a response within a week
or so, please send a
+kind reminder to the security team. We will usually let you know the CVE
number that will be assigned
+to the issue and the severity of the issue as well as release the issue is
scheduled to be fixed
+after we assess the issue (which might take longer or shorter time depending
on the issue complexity and
+potential impact, severity, whether we want to address a whole class issues in
a single fix and a number
+of other factors). You should subscribe and monitor the
`[email protected]` mailing
+list or `[email protected]` to see the announcement of the security issue -
including the CVE number
+and the severity of the issue once the issue is fixed and release is public.
Note that the
+`[email protected]` mailing list is moderated, and it might take a few hours
for the announcement to
+appear there, so `[email protected]` is the best place to monitor for
the announcement.
+
+Security issues in Airflow are handled by the Airflow Security Team. Details
about the Airflow Security
+Team and how members of it are chosen can be found in the [Contributing
documentation](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#security-team).
+
+### Does CVE in Airflow Providers impact Airflow core package ?
Airflow core package is released separately from provider packages. While
Airflow comes with ``constraints``
which describe which version of providers have been tested when the version of
Airflow was released, the
-users of Airflow are advised to install providers independently from Airflow
core when they want to apply
+users of Airflow are advised to install providers independently of Airflow
core when they want to apply
security fixes found and released in providers. Therefore, the issues found
and fixed in providers do
not apply to the Airflow core package. There are also Airflow providers
released by 3rd-parties, but the
Airflow community is not responsible for releasing and announcing security
vulnerabilities in them, this
is handled entirely by the 3rd-parties that release their own providers.
-## How are security issues handled in Airflow
+### Where do I find more information about Airflow Security ?
-Security issues in Airflow are handled by the Airflow Security Team. Details
about the Airflow Security Team and how members of it are chosen can be found
in the [Contributing
documentation](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#security-team).
+If you wish to know more about the ASF security process,
+the [ASF Security team's page](https://www.apache.org/security/) describes
+how vulnerability reports are handled in general by all ASF projects, and
includes PGP keys if
+you wish to use them when you report the issues.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 09d8bd279a..80d5136097 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -33,7 +33,7 @@ repos:
- id: doctoc
name: Add TOC for Markdown and RST files
files:
-
^CONTRIBUTING\.md$|^README\.md$|^UPDATING.*\.md$|^chart/UPDATING.*\.md$|^dev/.*\.md$|^dev/.*\.rst$
+
^CONTRIBUTING\.md$|^README\.md$|^UPDATING.*\.md$|^chart/UPDATING.*\.md$|^dev/.*\.md$|^dev/.*\.rst$|^.github/.*\.md
exclude: ^.*/.*_vendor/
args:
- "--maxlevel"
