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

Marking items un-satisfied #77

Open
dylanz opened this issue Nov 4, 2019 · 3 comments
Open

Marking items un-satisfied #77

dylanz opened this issue Nov 4, 2019 · 3 comments

Comments

@dylanz
Copy link

dylanz commented Nov 4, 2019

I'm just starting here, and after initializing the project, I want to mark all items unsatisfied until I've had a chance to go through them all. I see there's a .Satisfied that determines if it is or not, but I can't tell for the life of me how to set that true or not. I guess my questions are:

  • How do I mark all the items as unsatisfied?
  • Would a good first step in the process be to manually mark things as unsatisfied (or by default, not sure), then to work through the list?
@joepurdy
Copy link

@dylanz I opened a PR that would default newly initialized compliance projects to show controls as un-satisfied.

Just as a heads up in case my PR doesn't land in comply or you want to manually un-satisfy controls you can simply comment out the satisfies front-matter on all the policies and narratives that you haven't implemented.

For example, instead of this:

name: Organizational Narrative
acronym: ON
satisfies:
  TSC:
    - CC1.2
    - CC1.3
    - CC1.4
    - CC1.5
    - CC3.1
    - CC3.2
    - CC3.3
majorRevisions:
  - date: Jun 1 2018
    comment: Initial document
---

Comment out the satisfies front matter like this:

name: Organizational Narrative
acronym: ON
satisfies:
<!-- satisfies:
  TSC:
    - CC1.2
    - CC1.3
    - CC1.4
    - CC1.5
    - CC3.1
    - CC3.2
    - CC3.3 -->
majorRevisions:
  - date: Jun 1 2018
    comment: Initial document
---

This will ensure comply doesn't register these criteria as satisfied. Once the policy is implemented you can uncomment the satisfies front matter and that will mark controls as satisfied.

@dylanz
Copy link
Author

dylanz commented Dec 18, 2019

@joepurdy Boom! This is fantastic. Thanks for the PR and the heads up!

@joepurdy
Copy link

@dylanz just a heads up. I closed #79 in favor of an alternate approach in #85.

Basically I think since this mainly impacts new users that are still implementing the default template policies it's better to just make those default templates generate with the satisfies frontmatter commented out. That has the same effect as showing controls unsatisfied as adding new frontmatter like live: true|false.

And by making the change to the default templates it only impacts newly generated Comply programs. Had we merged #79 it would have led to a breaking change release where projects that weren't using the new live frontmatter suddenly register all controls as unsatisfied.

Just wanted to give you a heads up in case you ended up using a forked version of Comply based off my original PR that added the live frontmatter. I'd recommend switching back to upstream Comply and adopting the pattern of commenting out the satisfies frontmatter until a policy is customized and implemented in your organization.

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