Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Nov 20, 2023. It is now read-only.

add remediation variable to integration variables #291

Conversation

woutersm
Copy link
Member

@woutersm woutersm commented Feb 3, 2021

Added the remediation variable to the list of integration variables.

@@ -32,3 +32,4 @@ Variable | Details | Example
`{{title}}` | The title of the report. |
`{{vulnerability_information}}` | The description of the vulnerability in the report. |
`{{weakness}}` | The CWE weakness of the report. |
`{{remediation_advice}}` | The remediation advice created from the custom remediation advice with a fall back to the automated remediation advice. |
Copy link
Member

Choose a reason for hiding this comment

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

Can you add it in alphabetical ordering?

Copy link
Member

Choose a reason for hiding this comment

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

Do we have more documentation on remediation advice? Perhaps link it from the description.

Copy link
Member Author

Choose a reason for hiding this comment

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

This is the only place where the remediation advice is described in the docs. But I don't think it's adding any value to link to that page, what do you think?

Copy link
Member

Choose a reason for hiding this comment

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

Ah no, that doesn't add any value indeed :)

@stacyspiva
Copy link
Contributor

@martijnrusschen @woutersm The description repeats "remediation advice" 3 times, which isn't very ideal:

The remediation advice created from the custom remediation advice with a fall back to the automated remediation advice.

Can we simplify the description to something like: The customized remediation advice from MITRE. This will fall back to the automated remediation advice if there is no guidance from MITRE.

@stacyspiva stacyspiva merged commit b8ea13c into Hacker0x01:master Feb 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants