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

suggest new status codes #80

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shawndwells
Copy link
Member

Adds new status codes:

inherited: such as physical controls being inherited when running on a FedRAMP IaaS

not applicable: added to align with FedRAMP documentation options, shows control is N/A. For example program management controls for a technology component.

unknown: suggest as default return for elements missing implementation_status

unsatisfied: knowingly unable to meet control

@shawndwells
Copy link
Member Author

at some point there are many new schema elements to redo/introduce, but making this minor change seemed reasonable for a v3.1.1 schema update

@aegershman
Copy link

I realize this particular thread might be dead, but I'd like to suggest that if a status code of "not applicable" is introduced, I'd like to +1 suggest the use of either not_applicable or not-applicable, i.e. using a separator which isn't a space.

It's more conductive to automation which would use the status codes somehow (I don't have an example usecase, but it's generally valuable to be skeptical of whitespace), less to futz with for systems that implement spaces differently... And most importantly it's more ✨ aesthetically pleasing

that's all 👍

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.

2 participants