-
Notifications
You must be signed in to change notification settings - Fork 53
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
add basic security pages #985
Merged
Merged
Changes from all commits
Commits
Show all changes
6 commits
Select commit
Hold shift + click to select a range
74ca88c
add basic security pages
matt257 b79b0bd
Update security.rst
matt257 3c6ccdd
Update security.rst
matt257 b1bf7e5
Merge branch 'latest' into add-basic-security-pages
matt257 7cc3432
Update index.rst
matt257 f273db9
updated spacing in customization.rst so \'rake build\' works
matt257 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,50 @@ | ||
.. _security: | ||
|
||
Security | ||
======== | ||
|
||
Introduction | ||
------------ | ||
This document details the security framework for Open OnDemand, providing essential information that administrators need to know for secure deployment and operation. | ||
|
||
Considerations | ||
-------------- | ||
This section outlines key security advantages and areas for vigilance within the Open OnDemand environment. | ||
|
||
Advantages | ||
^^^^^^^^^^ | ||
|
||
- **Per-user Nginx (PUN) Architecture**: By running individual Nginx instances per user, Open OnDemand ensures that actions such as file access are conducted under user-specific non-root privileges, which isolates processes and enhances security. | ||
|
||
- **Robust Authentication**: Open OnDemand supports various authentication methods. It particularly emphasizes secure protocols over less secure options like Basic or LDAP authentication, reinforcing its commitment to high security standards. | ||
|
||
Limitations | ||
^^^^^^^^^^^ | ||
|
||
- **HTTP Traffic to Origin Servers**: Traffic to backend services, including computational resources like Jupyter servers, is currently over HTTP, which is unencrypted. Plans are underway to upgrade this to HTTPS to ensure encryption of data in transit, thereby bolstering security. | ||
|
||
Security Controls | ||
----------------- | ||
|
||
- **Monitoring and Logging**: Comprehensive logging mechanisms are integral for security audits and incident response. Detailed guidelines and settings for these features can be found at :ref:`logging`. | ||
|
||
- **Vulnerability Management**: Active management of security weaknesses includes regular updates and patches. Detailed processes and current security advisories are available at :ref:`vulnerability-management`. | ||
|
||
- **Security Audits**: The platform undergoes periodic security audits by Trusted CI, the NSF Cybersecurity Center of Excellence. Summaries of these audits are available, with the latest report accessible `here <https://openondemand.org/sites/default/files/documents/Trusted%20CI%20Open%20OnDemand%20Engagement%20Final%20Report%20-%20REDACTED%20FOR%20PUBLIC%20RELEASE%20210712_0.pdf>`_. | ||
|
||
Conclusion | ||
---------- | ||
Maintaining a secure and robust operational environment is critical for the success of Open OnDemand. Administrators are encouraged to implement the security practices recommended in this guide and to regularly review security settings and updates. | ||
|
||
|
||
Relevant References | ||
------------------- | ||
|
||
.. toctree:: | ||
:maxdepth: 2 | ||
:caption: Security Topics | ||
|
||
security/vulnerability-management | ||
authentication/overview | ||
how-tos/monitoring/logging | ||
customizations | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,42 @@ | ||
.. _vulnerability-management: | ||
|
||
Vulnerability Management | ||
======================== | ||
|
||
Introduction | ||
------------ | ||
|
||
Vulnerability management is a critical component of the security strategy for Open OnDemand. This document outlines the procedures for reporting and managing vulnerabilities. | ||
|
||
Reporting a Vulnerability | ||
------------------------- | ||
|
||
If you have security concerns or think you have found a vulnerability, please submit a private report by visiting the 'Security' section of our GitHub located at `GitHub Open OnDemand Security <https://github.com/OSC/ondemand/security/>`_ and clicking 'Report a vulnerability'. | ||
|
||
For direct inquiries or issues in submitting a report, contact the core project team via email at [email protected]. | ||
|
||
Disclosure Policy | ||
----------------- | ||
|
||
- Upon reporting, you will receive a response within hours, acknowledging the receipt of the report. | ||
- A primary handler from the team will be assigned to coordinate the fix and release process: | ||
- Confirm the problem and determine the affected versions (1-2 days). | ||
- Audit code to find any potential similar problems. | ||
- Prepare fixes for all releases still under maintenance and release as soon as possible. | ||
|
||
Comments on Policy | ||
------------------ | ||
|
||
Suggestions to improve this process can be made via submitting a ticket, opening a Discourse topic, or a pull request. | ||
|
||
Security Audits | ||
--------------- | ||
|
||
Open OnDemand has been audited several times by Trusted CI, the NSF Cybersecurity Center of Excellence. The latest engagement report is available `here <https://openondemand.org/sites/default/files/documents/Trusted%20CI%20Open%20OnDemand%20Engagement%20Final%20Report%20-%20REDACTED%20FOR%20PUBLIC%20RELEASE%20210712_0.pdf>`__. These audits have helped shape the security landscape of the platform and contribute to its ongoing security enhancements. | ||
|
||
Conclusion | ||
---------- | ||
|
||
Effective vulnerability management is crucial for maintaining the security and integrity of Open OnDemand. Users and contributors play a vital role in this process by reporting potential security vulnerabilities through GitHub, ensuring the platform's continued safety. | ||
|
||
.. note:: For details on the specific vulnerability management steps, please see the GitHub repository guidelines or the security policies linked above. |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I'm not in love with blink links. I mean - yes we log, but what's the context here, is that good or bad? Does it need updated, does the default work well? Yes we have customizations, but are they secure/insecure?
I think what we need is more up the page where the other considerations are. Something like
It's stuff like that - describing some behaviors that may be unwanted and specific links on how to control that.
We can keep this ToC here - but I feel like we need to be more direct with the reader and not just hope they explore and find the relevant info (and make the relevant connections) themselves.
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.
the context is logging works as 'how-tos/monitoring/logging' describes.
I don't think I can tell people if a feature is good or bad. I think I can only remove doubt about what a feature is, so people can be convinced whatever judgements they make (if they make any) are just.
If there are specific situations you think should be documented I can open up issues and work on writing something up for them and getting them added.
i'll try to update the 'considerations' from your comment