From 34160be6a0620608ae5ea12961faf89ecdaa36e6 Mon Sep 17 00:00:00 2001
From: nusantara-self <15647296+nusantara-self@users.noreply.github.com>
Date: Fri, 7 Mar 2025 11:34:43 +0800
Subject: [PATCH] Visual improvements

---
 .../MSEntraID_GetSignIns_1_0/long.html        | 67 +++++++++----------
 1 file changed, 32 insertions(+), 35 deletions(-)

diff --git a/thehive-templates/MSEntraID_GetSignIns_1_0/long.html b/thehive-templates/MSEntraID_GetSignIns_1_0/long.html
index d9f508b50..d6b097a89 100644
--- a/thehive-templates/MSEntraID_GetSignIns_1_0/long.html
+++ b/thehive-templates/MSEntraID_GetSignIns_1_0/long.html
@@ -1,74 +1,74 @@
+<!-- When no sign-ins found -->
 <div class="panel panel-success" ng-if="success && content.signIns.length == 0">
     <div class="panel-heading">
-        Microsoft Entra ID Sign Ins
+        <i class="fa fa-info-circle"></i> Microsoft Entra ID Sign Ins
     </div>
-
     <div class="panel-body">
-        Analyzers searched for: {{content.filterParameters}}
+        <p><i class="fa fa-filter"></i> Analyzers searched for: {{content.filterParameters}}</p>
     </div>
 </div>
 
+<!-- When sign-ins are available -->
 <div class="panel panel-primary" ng-if="success && content.signIns.length > 0">
     <div class="panel-heading">
-        Microsoft Entra ID Sign Ins
+        <i class="fa fa-sign-in-alt"></i> Microsoft Entra ID Sign Ins
     </div>
-
     <div class="panel-body">
         <table class="table table-striped">
             <thead>
                 <tr>
-                    <th>SignIn ID</th>
-                    <th>Time</th>
-                    <th>Status</th>
-                    <th>IP</th>
-                    <th>App Name</th>
-                    <th>Risk</th>
-                    <th>Device ID</th>
-                    <th>Device Name</th>
-                    <th>Device OS</th>
-                    <th>Cond'l Access</th>
-                    <th>Location</th>
+                    <th><i class="fa fa-id-badge"></i> SignIn ID</th>
+                    <th><i class="fa fa-clock"></i> Time</th>
+                    <th><i class="fa fa-info"></i> Status</th>
+                    <th><i class="fa fa-network-wired"></i> IP</th>
+                    <th><i class="fa fa-th"></i> App Name</th>
+                    <th><i class="fa fa-exclamation-triangle"></i> Risk</th>
+                    <th><i class="fa fa-desktop"></i> Device ID</th>
+                    <th><i class="fa fa-mobile-alt"></i> Device Name</th>
+                    <th><i class="fa fa-laptop"></i> Device OS</th>
+                    <th><i class="fa fa-shield-alt"></i> Cond'l Access</th>
+                    <th><i class="fa fa-map-marker-alt"></i> Location</th>
                 </tr>
             </thead>
             <tbody>
                 <!-- <tr ng-repeat="r in content.signIns" ng-style="{ 'background-color': r.locationDetails.countryOrRegion == 'US' && r.locationDetails.state != 'STATE' ? '#ebc27c' : r.locationDetails.countryOrRegion != 'COUNTRY' ? '#eb807c' : 'inherit' }"> -->
                 <tr ng-repeat="r in content.signIns">
-                    <td title={{r.id}}>{{r.id | limitTo: 8}}</td>
+                    <td title="{{r.id}}">{{r.id | limitTo: 8}}</td>
                     <td>{{r.basicDetails.signInTime}}</td>
-                    <td title={{r.basicDetails.result}}>{{r.basicDetails.result | limitTo: 7}}</td>
-                    <td title={{r.basicDetails.ip}} ng-if="r.basicDetails.ip.length > 15">IPv6</td>
+                    <td title="{{r.basicDetails.result}}">{{r.basicDetails.result | limitTo: 7}}</td>
+                    <td title="{{r.basicDetails.ip}}" ng-if="r.basicDetails.ip.length > 15">IPv6</td>
                     <td ng-if="r.basicDetails.ip.length <= 14">{{r.basicDetails.ip}}</td>
                     <td>{{r.basicDetails.appName}}</td>
                     <td>{{r.basicDetails.riskLevel}}</td>
                     <td ng-if="r.deviceDetails.id == 'Not Available'">Not Available</td>
-                    <td ng-if="r.deviceDetails.id != 'Not Available'" title={{r.deviceDetails.id}}>{{r.deviceDetails.id | limitTo: 8}}</td>
+                    <td ng-if="r.deviceDetails.id != 'Not Available'" title="{{r.deviceDetails.id}}">{{r.deviceDetails.id | limitTo: 8}}</td>
                     <td>{{r.deviceDetails.deviceName}}</td>
                     <td>{{r.deviceDetails.operatingSystem}}</td>
                     <td ng-if="r.appliedConditionalAccessPolicies == 'None'">No</td>
                     <td ng-if="r.appliedConditionalAccessPolicies != 'None'">Yes</td>
                     <td>{{r.locationDetails.city}}, {{r.locationDetails.state}}, {{r.locationDetails.countryOrRegion}}</td>
-
                 </tr>
             </tbody>
         </table>
     </div>
 </div>
+
+<!-- Expanded Information -->
 <div class="panel panel-primary" ng-if="success && content.signIns.length > 0">
     <div class="panel-heading">
-        Expanded Information
+        <i class="fa fa-info-circle"></i> Expanded Information
     </div>
-
     <div class="panel-body">
         <table class="table table-striped">
             <thead>
                 <tr>
-                    <th>SignIn ID</th>
-                    <th>IPv6</th>
-                    <th>App Name</th>
-                    <th>Client App</th>
-                    <th>Resource Name</th>
-                    <th>Applied CAPs</th>
-                    <th>Device ID</th>
+                    <th><i class="fa fa-id-badge"></i> SignIn ID</th>
+                    <th><i class="fa fa-network-wired"></i> IPv6</th>
+                    <th><i class="fa fa-th"></i> App Name</th>
+                    <th><i class="fa fa-cubes"></i> Client App</th>
+                    <th><i class="fa fa-book"></i> Resource Name</th>
+                    <th><i class="fa fa-check-square"></i> Applied CAPs</th>
+                    <th><i class="fa fa-desktop"></i> Device ID</th>
                 </tr>
             </thead>
             <tbody>
@@ -85,19 +85,16 @@
             </tbody>
         </table>
     </div>
-    
 </div>
 
-        
-
 <!-- General error  -->
 <div class="panel panel-danger" ng-if="!success">
     <div class="panel-heading">
-        <strong>{{(artifact.data || artifact.attachment.name) | fang}}</strong>
+        <i class="fa fa-warning"></i> <strong>{{(artifact.data || artifact.attachment.name) | fang}}</strong>
     </div>
     <div class="panel-body">
         <dl class="dl-horizontal" ng-if="content.errorMessage">
-            <dt><i class="fa fa-warning"></i> GetAzureSignIns:</dt>
+            <dt><i class="fa fa-warning"></i> MSEntraID_GetSignIns:</dt>
             <dd class="wrap">{{content.errorMessage}}</dd>
         </dl>
     </div>