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

Added SolarWinds Orion fingerprints #308

Merged
merged 2 commits into from
Jan 20, 2021
Merged

Added SolarWinds Orion fingerprints #308

merged 2 commits into from
Jan 20, 2021

Conversation

scopedsecurity
Copy link
Contributor

Description

Added SolarWinds Orion fingerprints in html_title.xml and favicons.xml

<example>53317933c27890ae9218697ecc0e97d9</example>
<param pos="0" name="service.vendor" value="SolarWinds"/>
<param pos="0" name="service.product" value="Orion"/>
<param pos="0" name="service.certainty" value="0.5"/>
Copy link

Choose a reason for hiding this comment

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

Should we add the CPE for both of these fingerprints?
<param pos="0" name="service.cpe23" value="cpe:/a:solarwinds:orion_platform:-"/>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've updated both with the CPEs.

@tsellers-r7
Copy link
Contributor

I'll try to land this later today. I expect that the automatic CPE generation ( https://github.com/rapid7/recog/blob/master/CONTRIBUTING.md#updating-cpes ) will fail so we will either change the name of the product from Orion to Orion Platform or add a mapping here:

https://github.com/rapid7/recog/blob/master/cpe-remap.yaml

@tsellers-r7
Copy link
Contributor

@scopedsecurity - Landing this now, thanks for the contribution. As a note, I'll likely change Orion to Orion Platform to match the CPE.

@tsellers-r7 tsellers-r7 merged commit 5e1b207 into rapid7:master Jan 20, 2021
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.

3 participants