Skip to content

Commit 78c897d

Browse files
committed
rework the vulnerability data popover
Signed-off-by: tdruez <tdruez@aboutcode.org>
1 parent af8df34 commit 78c897d

3 files changed

Lines changed: 40 additions & 7 deletions

File tree

Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
<div>
2+
{% if vulnerability.aliases %}
3+
<div class='mb-2'>
4+
<strong>Aliases</strong>
5+
<div>
6+
{% for alias in vulnerability.aliases %}
7+
<div>
8+
{% if alias|slice:':3' == 'CVE' %}
9+
<a href='https://nvd.nist.gov/vuln/detail/{{ alias }}' target='_blank'>{{ alias }}</a>
10+
{% elif alias|slice:':4' == 'GHSA' %}
11+
<a href='https://github.com/advisories/{{ alias }}' target='_blank'>{{ alias }}</a>
12+
{% elif alias|slice:':3' == 'NPM' %}
13+
<a href='https://github.com/nodejs/security-wg/blob/main/vuln/npm/{{ alias|slice:"4:" }}.json' target='_blank'>{{ alias }}</a>
14+
{% else %}
15+
{{ alias }}
16+
{% endif %}
17+
</div>
18+
{% endfor %}
19+
</div>
20+
</div>
21+
{% endif %}
22+
{% if vulnerability.summary %}
23+
<div class='mb-2'>
24+
<strong>Summary</strong>
25+
<div>{{ vulnerability.summary|linebreaksbr }}</div>
26+
</div>
27+
{% endif %}
28+
{% if vulnerability.exploitability %}
29+
<div class='mb-1'><strong>Exploitability:</strong> {{ vulnerability.get_exploitability_display }}</div>
30+
{% endif %}
31+
{% if vulnerability.risk_score %}
32+
<div><strong>Risk:</strong> {{ vulnerability.risk_score }}</div>
33+
{% endif %}
34+
</div>

dejacode/static/css/dejacode_bootstrap.css

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1075,3 +1075,7 @@ select.bootstrap-select-filter {
10751075
position: absolute;
10761076
z-index: 1;
10771077
}
1078+
.popover.wide-popover {
1079+
--bs-popover-max-width: 420px;
1080+
max-width: 420px;
1081+
}

product_portfolio/templates/product_portfolio/tabs/tab_packages_vulnerabilities.html

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@
4848
{% else %}
4949
{{ vulnerability.advisory_id }}
5050
{% endif %}
51-
{% if vulnerability.summary %}
52-
<span class="float-end" data-bs-toggle="popover" data-bs-placement="right" data-bs-trigger="hover focus" data-bs-html="true" data-bs-content="{{ vulnerability.summary|linebreaksbr }}">
51+
{% if vulnerability.summary or vulnerability.aliases or vulnerability.exploitability or vulnerability.risk_score %}
52+
<span class="float-end" data-bs-toggle="popover" data-bs-placement="right" data-bs-trigger="hover focus" data-bs-html="true" data-bs-custom-class="wide-popover" data-bs-content="{% include 'component_catalog/includes/vulnerability_info_popover.html' with vulnerability=vulnerability only %}">
5353
<i class="fa-solid fa-circle-info text-muted"></i>
5454
</span>
5555
{% endif %}
@@ -68,11 +68,6 @@
6868
</span>
6969
{% endif %}
7070
</div>
71-
{% if vulnerability.aliases %}
72-
<div class="mt-2">
73-
{% include 'component_catalog/includes/vulnerability_aliases.html' with aliases=vulnerability.aliases only %}
74-
</div>
75-
{% endif %}
7671
</td>
7772
{% if has_triage_rulesets %}
7873
<td>

0 commit comments

Comments
 (0)