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 <ja...@potiuk.com> 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 [secur...@airflow.apache.org](mailto:secur...@airflow.apache.org). +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 `us...@airflow.apache.org` mailing +list or `annou...@apache.org` 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 +`annou...@apache.org` mailing list is moderated, and it might take a few hours for the announcement to +appear there, so `us...@airflow.apache.org` 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"