Skip to content

Conversation

@IlonaShishov
Copy link
Collaborator

This PR introduces vex generation support.
Including changes to support new output schema containing vex document.
Displays and provide a download option for the generated VEX document.

Copy link
Collaborator

@zvigrinberg zvigrinberg left a comment

Choose a reason for hiding this comment

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

Hi @IlonaShishov Very good job!.
I Have few minor comments, PTAL.

Thanks!

List<Document> outputDocs = objectMapper.readValue(report.get("output").toPrettyString(),
new TypeReference<List<Document>>() {
Document outputDoc = objectMapper.readValue(report.get("output").toPrettyString(),
new TypeReference<Document>() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

@IlonaShishov TypeReference is usually for deserializing json arrays of objects into Collections that contains POJOs, In that case, you can use POJO Document.class directly for de-serialization.

quarkus.rest-client.morpheus.url=https://agent-morpheus:8080/generate
%dev.quarkus.rest-client.morpheus.url=http://localhost:26466/generate
quarkus.rest-client.component-syncer.url=http://job-sink.knative-eventing.svc.cluster.local/exploit-iq-nat/component-syncer
quarkus.rest-client.component-syncer.url=http://job-sink.knative-eventing.svc.cluster.local/${NAMESPACE}/component-syncer
Copy link
Collaborator

Choose a reason for hiding this comment

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

Good catch @IlonaShishov , putting it as exploit-iq-nat by default is not flexible and requires manual update in config all the time.
The best way and easiest way to address this is using Downward API ( how we currently do it in the main deployment located in the vulnerability-analysis repo. in the client deployment there).
Can you please take it from here:

https://github.com/RHEcosystemAppEng/vulnerability-analysis/blob/main/kustomize/base/exploit_iq_client.yaml#L67-L70

And copy it to here for the sake of good order:

https://github.com/RHEcosystemAppEng/agent-morpheus-client/blob/main/deploy/agent_morpheus_client.yaml#L77

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