-
Notifications
You must be signed in to change notification settings - Fork 85
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
Add a way to include other files besides components, standards, and certifications. #131
Comments
@jcscottiii Do you have a branch for this? No pressure...just curious. |
Example: Search for "diagram" in https://compliance.cloud.gov/system_documentation/system-description.html |
Let's investigate if there's a workaround for this that requires little to no code/example changes, e.g. "put all required assets in the |
There's two parts missing:
|
There are 2 cases for inclusion that happen. 1) Inclusion of files that are next to the component.yaml and 2) Inclusion of completely different folders (e.g. 1) Inclusion of files local to component.yamlCurrently, given a component (e.g.
Both files will be included into the 2) Inclusion of completely different foldersThis doesn't currently happen. This following line uses BDD but the link would be broken. The proposed solution is to add an Then when it comes to rendering, it will do like the old python code and prepend an ConclusionWell, since I'm putting files in In the end, part of your opencontrol.yaml will look like: includes:
- BDD # path will be "includes/BDD"
- UAA/auth-diagram.png # path will be "includes/auth-diagram.png" Why be so explicit about what you're including and not letting things next to component.yamls be automatically copied into the |
@jcscottiii Thoughts about my comment above? I want to be very careful about us over-promising anything with this feature. With Word docs as a common format for system security plans, we can't (easily) do anything with included files. Even embedding images could be tricky. Therefore, it feels safer to me to punt on it (for now) and tell people to use external URLs. Thoughts? |
@afeld I agree that the latter of the two would work and be the cleanest. (The former would require code changes because inherited markdowns are not included right now) It brings about another problem about hosting those assets but you know ¯_(ツ)_/¯ not on us now lol. But a great alternative would just have people host it in their repos and point to the raw links. |
Yeah, that seems like the easiest thing for now, though I could be convinced that we should try and figure this out for images, at least. How do you want to proceed? |
I created a new issue to research it: opencontrol/fedramp-templater#19 We need to create a how-to-use or FAQ for the component.yaml. I can create this. |
Another relevant issue: cloud-gov/compliance#166. @brittag mentions there that architecture diagrams aren't readable shoved into a table cell in a Word doc, so these would need to be provided as an attachment or a link anyway. |
Currently, there's no way to include other files when getting all components, certifications, and standards. Would like to have an
includes
section in the yaml for misc data.This will fix some broken links in the gitbook
The text was updated successfully, but these errors were encountered: