-
Notifications
You must be signed in to change notification settings - Fork 169
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
Missing documentation for conditions #28
Comments
@shambho Does your commit 3 days ago resolve this issue? If so, let's close this one. |
@colesmj nope. Commit from 3 days ago was about adding some cosmetic changes to tm.py, Readme and pytm.py. Agree, we should have more documentation on the properties available in each Class. |
#91 documents a very small subset of attributes. |
I would like to second this issue report: flags such as "sourceFiles" are not clear to the average user/reader, without explanation. |
+1 Being new to threat modeling, some of these are quite hard to understand. Some can be figured out with a google search, like |
Some properties/conditions are obvious and don't really need documentation, such as protocol or isEncrypted. But other properties may not be obvious to some, such as Dataflow.authenticatedWith - with what?
The text was updated successfully, but these errors were encountered: