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

get_control_response missing "This System" component #1833

Open
rahearn opened this issue Mar 11, 2025 · 0 comments · May be fixed by #1834
Open

get_control_response missing "This System" component #1833

rahearn opened this issue Mar 11, 2025 · 0 comments · May be fixed by #1834
Labels
bug Something isn't working
Milestone

Comments

@rahearn
Copy link
Contributor

rahearn commented Mar 11, 2025

Describe the bug

For controls with statements, the "This System" implementation statements are missing from the individual parts.

This appears to be a regression introduced in #1358

To Reproduce

Steps to reproduce the behavior:

  1. Create a system security plan with a control with multiple parts (such as SC-7)
  2. Use trestle author jinja in combination with ssp_md_writer.get_control_response to create a markdown SSPP
  3. See that the implementation statements for each individual part are missing for the This System component (though they'll be present for any other components)

Expected behavior

I expected that This System implementation statements are included for all parts of the control.

Screenshots / Logs.

If applicable, add screenshots to help explain your problem.

Environment

  • OS: [e.g. iOS] Docker/Debian
  • Python version: 3.11.11
  • Installed packages: compliance-trestle 3.8.0
@jpower432 jpower432 added the bug Something isn't working label Mar 11, 2025
@jpower432 jpower432 moved this from New to In review in Trestle Roadmap Mar 11, 2025
@jpower432 jpower432 added this to the v3.8.1 milestone Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: In review
Development

Successfully merging a pull request may close this issue.

2 participants