Skip to content

Commit 3481175

Browse files
authored
feat(admin): display an inspector link if found (#17931)
* feat(admin): display an inspector link if found If the inbound report contained an inspector link, display it on the project list for speedy access. Signed-off-by: Mike Fiedler <[email protected]> * fix(admin): use correct property for detail iframe Things were renamed while this was fresh, and the display part never got the fix. Signed-off-by: Mike Fiedler <[email protected]> --------- Signed-off-by: Mike Fiedler <[email protected]>
1 parent aaa7a72 commit 3481175

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

warehouse/admin/templates/admin/malware_reports/detail.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,8 +102,8 @@ <h3 class="card-title">
102102
<div class="card-body">
103103
<div class="embed-responsive embed-responsive-16by9">
104104
{# If we can deep-link to the inspector spot, do so #}
105-
{% if report.payload.get("inspector_link") %}
106-
{% set inspector_url = report.payload.get("inspector_link") %}
105+
{% if report.payload.get("inspector_url") %}
106+
{% set inspector_url = report.payload.get("inspector_url") %}
107107
{% else %}
108108
{% if report.related %}
109109
{% set inspector_url = "https://inspector.pypi.io/project/" + report.related.name %}

warehouse/admin/templates/admin/malware_reports/project_list.html

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,14 @@ <h3 class="timeline-header">
5656
<dd class="col-sm-10">
5757
{{ report.summary }}
5858
</dd>
59+
{% if report.payload.get("inspector_url") %}
60+
<dt class="col-sm-2">Inspector</dt>
61+
<dd class="col-sm-10">
62+
<a href="{{ report.payload.get("inspector_url") }}" target="_blank">Inspector Link
63+
<i class="fa-solid fa-arrow-up-right-from-square"></i>
64+
</a>
65+
</dd>
66+
{% endif %}
5967
{# TODO: Decide how to pretty this up when we have more data #}
6068
{% if report.actions %}
6169
<dt class="col-sm-2">Actions</dt>

0 commit comments

Comments
 (0)