Skip to content
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

pySCG: updated to scope addressing Davids feedback, fixed formatting #710

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

myteron
Copy link
Contributor

@myteron myteron commented Dec 17, 2024

As per SIG meeting Dec 16, addressing our unclear scope in our introduction of the main readme.md.
Removed some duplicated and long winded wording.
Fixed links and references.
Fixed formatting issues.

…ted wording and fixed formatting

Signed-off-by: Helge Wehder <helge.wehder@ericsson.com>
Signed-off-by: Helge Wehder <helge.wehder@ericsson.com>
Signed-off-by: Helge Wehder <helge.wehder@ericsson.com>
Common Weakness Enamurator (CWE) [Pillar Weakness](https://cwe.mitre.org/documents/glossary/#Pillar%20Weakness) [mitre.org 2023].

Some rules only contain code examples, documentation will follow.
Python modules outside of the _Python Module Index_ [[Python 2023](https://docs.python.org/3.9/py-modindex.html)], languages such as `HTML` and `JavaScript` are avoided.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Python modules outside of the _Python Module Index_ [[Python 2023](https://docs.python.org/3.9/py-modindex.html)], languages such as `HTML` and `JavaScript` are avoided.
Python modules outside of the _Python Module Index_ [[Python 2023](https://docs.python.org/3.9/py-modindex.html)] are not specifically covered by this document. We do sometimes note where outside modules are necessary to address a type of vulnerability, and encourage users of those modules to consult those modules' documentation.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The note about HTML & JavaScript isn't really the issue. The point is that this document only discusses the built-in modules. Presumably JSON would be in scope, because there's a built-in module for JSON.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@david-a-wheeler , the background story to this learning resource is that we have designers spending 40 hours going through a ton of text to learn secure coding in languages such as JAVA. Any chance we can shorten that sentence?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Python modules outside of the _Python Module Index_ [[Python 2023](https://docs.python.org/3.9/py-modindex.html)], languages such as `HTML` and `JavaScript` are avoided.
Python modules outside of the _Python Module Index_ [[Python 2023](https://docs.python.org/3.9/py-modindex.html)] are not specifically covered by this document.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about this? The issue isn't really HTML (there's an html module in Python's built-ins) - it's that this particular document only focuses on the built-ins.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much better thank you.

Since I am not native English speaking I struggle to understand the difference between "avoiding" and "not specifically covered". We try our best to only list "inclusions" in order to have ever expanding lists of "exclusions". Not sure how to best phrase that in this introduction.

We are missing a contribution document, I will get onto creating that. In it I would outline the following in regards to explaining why we avoid modules outside of the standard lib:

  • Code examples may end up as module recommendations "just use..."
  • Actual issue is not explained in code.
  • Don't want to end up writing module documentation, bug-fixes or workarounds.
  • Life Cycle related issues
  • Trust related issues

We also want to keep the main pySCG readme.md look "inviting" with a minimum of content and links in its introduction section.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants