From f3d9e2474d59ddbb5a7e80378e3a0901d11bc963 Mon Sep 17 00:00:00 2001 From: Stephan Kergomard Date: Tue, 2 Jul 2024 08:29:45 +0200 Subject: [PATCH 1/3] Docu: Shorten Lines in Security.md --- docs/development/security.md | 36 ++++++++++++++++++++++++++++-------- 1 file changed, 28 insertions(+), 8 deletions(-) diff --git a/docs/development/security.md b/docs/development/security.md index 03a2aed0f747..8c229c3fbc8b 100755 --- a/docs/development/security.md +++ b/docs/development/security.md @@ -8,31 +8,51 @@ ## Reporting Security Issues [//]: # (BEGIN Reporting) -Please make sure to understand, that treating security issues confidentially is required to keep ILIAS installations as safe as possible until the issue is fixed. +Please make sure to understand, that treating security issues confidentially is +required to keep ILIAS installations as safe as possible until the issue is fixed. + +Please follow the process described in detail below. You will receive an answer +from a member of the ILIAS security group about further steps. -Please follow the process described in detail below. You will receive an answer from a member of the ILIAS security group about further steps. **Do not file an issue in the bugtracker!** -1. Write an email to security@lists.ilias.de about your discovery, containing a description of the issue with the scenario in which the problem is triggered and a description of its implications. Please provide all necessary steps to reproduce the issue. We kindly ask you to withhold full disclosure of the issue until a fix is ready and the new release has been build and made available to everyone (full disclosure about 1 week after the new release is published). +1. Write an email to security@lists.ilias.de about your discovery, containing a +description of the issue with the scenario in which the problem is triggered and +a description of its implications. Please provide all necessary steps to reproduce +the issue. We kindly ask you to withhold full disclosure of the issue until a fix +is ready and the new release has been build and made available to everyone +(full disclosure about 1 week after the new release is published). 2. The Security Group will assign an issue manager. 3. The issue manager will look into the issue and try and reproduce the problem. -4. The issue manager will contact you on behalf of the ILIAS e.V. by email. We are grateful for any further help/information you can provide during the analysis and bugfixing process. -5. Depending on the severity and impact of the issue at hand, the developers will build a new release ASAP or continue with the default roadmap. -6. Optional: We are very interested in giving proper credit for your finding and your support for the project. If you want to, we can include your name and/or institution in our release notes. We will not publish your name or the name of your institution without your consent. +4. The issue manager will contact you on behalf of the ILIAS e.V. by email. +We are grateful for any further help/information you can provide during the +analysis and bugfixing process. +5. Depending on the severity and impact of the issue at hand, the developers will +build a new release ASAP or continue with the default roadmap. +6. Optional: We are very interested in giving proper credit for your finding and +your support for the project. If you want to, we can include your name and/or +institution in our release notes. We will not publish your name or the name of +your institution without your consent. [//]: # (END Reporting) ## Publishing Fixes for Security Issues [//]: # (BEGIN Publishing) -We are delighted when solutions are offered together with the initial report. Please be aware, however, that our repository in GitHub is also open to the general public: commits, commit-messages and pull-requests can be viewed by anyone. It is therefore also better in this case to get in touch with security@lists.ilias.de in order to discuss further steps with us. +We are delighted when solutions are offered together with the initial report. +Please be aware, however, that our repository in GitHub is also open to the +general public: commits, commit-messages and pull-requests can be viewed by +anyone. It is therefore also better in this case to get in touch with +security@lists.ilias.de in order to discuss further steps with us. [//]: # (END Publishing) ## Security Update Notifications [//]: # (BEGIN Notifications) -Please subscribe to our admin mailing list (ilias-admins@lists.ilias.de) to get notifications about security updates, updates in general and announcements for ILIAS server administrators. +Please subscribe to our admin mailing list (ilias-admins@lists.ilias.de) to get +notifications about security updates, updates in general and announcements for +ILIAS server administrators. [//]: # (END Notifications) From dfd14a6f04f3848389d2c8247cca2a62bcfacda7 Mon Sep 17 00:00:00 2001 From: Stephan Kergomard Date: Tue, 2 Jul 2024 09:22:00 +0200 Subject: [PATCH 2/3] Docu: Define Security Process --- docs/development/security.md | 35 +++++++++++++++++++++++++++++------ 1 file changed, 29 insertions(+), 6 deletions(-) diff --git a/docs/development/security.md b/docs/development/security.md index 8c229c3fbc8b..f2f9797aa349 100755 --- a/docs/development/security.md +++ b/docs/development/security.md @@ -38,21 +38,44 @@ your institution without your consent. ## Publishing Fixes for Security Issues [//]: # (BEGIN Publishing) - We are delighted when solutions are offered together with the initial report. -Please be aware, however, that our repository in GitHub is also open to the -general public: commits, commit-messages and pull-requests can be viewed by -anyone. It is therefore also better in this case to get in touch with -security@lists.ilias.de in order to discuss further steps with us. +Please follow the **Process for Fixing Security Issues** in order to not +unnecessarily endanger existing ILIAS installations. If you don't have access +to the corresponding issue in our internal security tracker please give access +to the corresponding patch files to the issue manager through a secure channel. [//]: # (END Publishing) +## Process for Fixing Security Issues +[//]: # (BEGIN Fixing) +The following process MUST be followed to hand in a fix for a security issue: +* Create one patch file per supported branch needing the fix. The patch file MUST +be named "_.patch" (e.g. "9_my-very-important-fix.patch"). +To create the patch use `git format-patch` with the option `--stdout` +(e.g. `git format-patch my-bug-fixing-branch --stdout > 9_my-very-important-fix.patch`) +to ensure that all commits are in a single patch file, if multiple commits are +needed to fix the issue. To make the application of the patches as easy as possible, +**fixes for security issues MUST not contain any unrelated changes**. +* Upload the patch files to the corresponding issue in the project "ILIAS Security" +on our issue tracker. If you would like to provide a patch for an issue, but do +not have access to the issue on our tracker please send the patch files to +the issue manager on a secure channel. +* Please provide a summary for the release notes and a few sentences describing +the fix either directly in the corresponding sections in the issue tracker or in +the communication. +* The fixes will be applied to the release branches by the release manager before +the release. + +[//]: # (END Fixing) + ## Security Update Notifications [//]: # (BEGIN Notifications) Please subscribe to our admin mailing list (ilias-admins@lists.ilias.de) to get notifications about security updates, updates in general and announcements for -ILIAS server administrators. +ILIAS server administrators. As a general rule ILIAS updates will be published +on the start of a week and will be announced in the middle of the previous week +on the mailing list. [//]: # (END Notifications) From f275c916b47bcc11b2717854630f8adeba610195 Mon Sep 17 00:00:00 2001 From: kergomard <13102171+kergomard@users.noreply.github.com> Date: Mon, 16 Sep 2024 14:41:04 +0200 Subject: [PATCH 3/3] Shorten E-Mail Address of Security-Group --- docs/development/security.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/development/security.md b/docs/development/security.md index f2f9797aa349..f167e415768d 100755 --- a/docs/development/security.md +++ b/docs/development/security.md @@ -1,4 +1,4 @@ -# ILIAS Security Group +#@ ILIAS Security Group ## Table of Contents * [Reporting Security Issues](#reporting-security-issues) @@ -16,7 +16,7 @@ from a member of the ILIAS security group about further steps. **Do not file an issue in the bugtracker!** -1. Write an email to security@lists.ilias.de about your discovery, containing a +1. Write an email to security@ilias.de about your discovery, containing a description of the issue with the scenario in which the problem is triggered and a description of its implications. Please provide all necessary steps to reproduce the issue. We kindly ask you to withhold full disclosure of the issue until a fix