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

Initial claims ontology #804

Merged
merged 10 commits into from
Nov 7, 2022
Merged

Initial claims ontology #804

merged 10 commits into from
Nov 7, 2022

Conversation

glguy
Copy link
Contributor

@glguy glguy commented Oct 14, 2022

No description provided.

@glguy glguy self-assigned this Oct 14, 2022
Copy link
Contributor

@bhatt111 bhatt111 left a comment

Choose a reason for hiding this comment

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

partiallySupports should be an attribute of PROPERTY_TYPE, not of PROPERTY, since that determination is made for the PROPERTY_TYPE. We need to think of PROPERTY_TYPE as a first class ENTITY, rather that just an "enumeration". We will still have predefined identifiers in SADL for PROPERTY_TYPE for he enumeration.

@glguy
Copy link
Contributor Author

glguy commented Oct 28, 2022

partiallySupports should be an attribute of PROPERTY_TYPE, not of PROPERTY, since that determination is made for the PROPERTY_TYPE. We need to think of PROPERTY_TYPE as a first class ENTITY, rather that just an "enumeration". We will still have predefined identifiers in SADL for PROPERTY_TYPE for he enumeration.

I'm expecting the PROPERTY_TYPEs to exist in the ontology before any OBJECTIVES are ever added. Would it make sense for OBJECTIVES to point to PROPERTY_TYPE, perhaps?

@bhatt111
Copy link
Contributor

bhatt111 commented Oct 28, 2022 via email

@glguy
Copy link
Contributor Author

glguy commented Oct 29, 2022

All of the claims types have been incorporated into the enumeration variant CSVs. I recommend using the [...] menu to select View File to get a pretty table view in the PR changed files tab.

We've instantiated the claims diagram with example bits from each of the contributing teams claims information.

Note that the "source" column in these CSV files are simply for reference as we iterate on this design. They are not going to be loaded into RACK itself.

Screen Shot 2022-10-28 at 4 58 46 PM
Screen Shot 2022-10-28 at 4 58 55 PM
Screen Shot 2022-10-28 at 4 59 08 PM

@glguy
Copy link
Contributor Author

glguy commented Oct 29, 2022

@bhatt111 we've moved partiallySupports over to a CLAIM per discussion today.

@glguy
Copy link
Contributor Author

glguy commented Nov 2, 2022

@bhatt111 I've incorporated your changes to the SRI overlay

@glguy glguy marked this pull request as ready for review November 2, 2022 16:35
@kityansiu
Copy link
Contributor

I just noticed that with the new claim enumeration types, we now have CWEs added as both PROPERTY_TYPES and MITRE-CWE. Is this really what we want? Should we remove one of them?

@bhatt111
Copy link
Contributor

bhatt111 commented Nov 7, 2022 via email

@glguy
Copy link
Contributor Author

glguy commented Nov 7, 2022

@kityansiu I agree that it's not great that the CWEs are duplicated.

Maybe another encoding would just be to have a single "CWE present in module" property and require the claim to use "addresses" to point the the actual MITRE_CWE instance.

@bhatt111
Copy link
Contributor

bhatt111 commented Nov 7, 2022

Extending what Eric said, another way (we are handling an analogous situation in the SRI team) is to have "CWE NOT present in module" as a PROPERTY_TYPE and then create individual instances of PROPERTY that would reference a particular CWE using “mitigates” or similar attribute.
Note: I suggest that properties should be stated in a positive way – i.e., property being satisfied is a good thing.
Devesh

@glguy glguy merged commit eb2f51e into master Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants