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

POAM related finding support, fixes #1120 #1478

Conversation

Compton-US
Copy link
Contributor

Committer Notes

To match the pattern of POAM for handling observations and risks, a related-findings assembly was added.

@Compton-US Compton-US changed the title [WIP] POAM related finding support, fixes #1120 POAM related finding support, fixes #1120 Sep 29, 2022
…to assessment common from assessment result metaschema.
@Compton-US
Copy link
Contributor Author

Compton-US commented Sep 29, 2022

Important note on this PR, since I realized I needed to adjust slightly:

There is the addition of the related-findings assembly, but I also needed to add the findings assembly, similar to the observations and risks assemblies in the POAM. As a part of this, the assessment-result model contained the findings assembly, so I moved that to assessment-common to support both the poam and assessment-result models.

@aj-stein-nist aj-stein-nist linked an issue Oct 17, 2022 that may be closed by this pull request
8 tasks
Copy link
Contributor

@david-waltermire david-waltermire left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@david-waltermire david-waltermire merged commit c0605b5 into usnistgov:develop Oct 17, 2022
aj-stein-nist added a commit that referenced this pull request Oct 18, 2022
* Add related finding assembly. #1120
* Add finding assembly to support related-findings, also moved finding to assessment common from assessment result metaschema.
* Accept suggestion to revise description.

Co-authored-by: Alexander Stein <[email protected]>
david-waltermire pushed a commit that referenced this pull request Oct 31, 2022
* Add related finding assembly. #1120
* Add finding assembly to support related-findings, also moved finding to assessment common from assessment result metaschema.
* Accept suggestion to revise description.

Co-authored-by: Alexander Stein <[email protected]>
aj-stein-nist added a commit to aj-stein-nist/OSCAL-forked that referenced this pull request Jan 10, 2023
* Add related finding assembly. usnistgov#1120
* Add finding assembly to support related-findings, also moved finding to assessment common from assessment result metaschema.
* Accept suggestion to revise description.

Co-authored-by: Alexander Stein <[email protected]>
aj-stein-nist added a commit to aj-stein-nist/OSCAL-forked that referenced this pull request Feb 6, 2023
* Add related finding assembly. usnistgov#1120
* Add finding assembly to support related-findings, also moved finding to assessment common from assessment result metaschema.
* Accept suggestion to revise description.

Co-authored-by: Alexander Stein <[email protected]>
aj-stein-nist added a commit to aj-stein-nist/OSCAL-forked that referenced this pull request Jun 29, 2023
* Add related finding assembly. usnistgov#1120
* Add finding assembly to support related-findings, also moved finding to assessment common from assessment result metaschema.
* Accept suggestion to revise description.

Co-authored-by: Alexander Stein <[email protected]>
aj-stein-nist added a commit to aj-stein-nist/OSCAL-forked that referenced this pull request Jul 10, 2023
* Add related finding assembly. usnistgov#1120
* Add finding assembly to support related-findings, also moved finding to assessment common from assessment result metaschema.
* Accept suggestion to revise description.

Co-authored-by: Alexander Stein <[email protected]>
aj-stein-nist added a commit to galtm/OSCAL that referenced this pull request Sep 28, 2023
* Add related finding assembly. usnistgov#1120
* Add finding assembly to support related-findings, also moved finding to assessment common from assessment result metaschema.
* Accept suggestion to revise description.

Co-authored-by: Alexander Stein <[email protected]>
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.

POAM Item should have a related-finding assembly
3 participants