.env.example

@@ -1,20 +1,2 @@
 DOMAIN=localhost
 DEMO=false
-
-STATIC_ROOT=/tmp/static/
-MEDIA_ROOT=/tmp/media/
-ALLOWED_HOSTS=localhost,localhost:8000,127.0.0.1,
-DOMAIN=localhost
-DEBUG=True
-EMAIL_HOST="mail.example.org"
-EMAIL_HOST_USER="fillme_noreply"
-EMAIL_HOST_PASSWORD="fillme_passwd"
-EMAIL_PORT=587
-EMAIL_USE_TLS=True
-EMAIL_BACKEND="django.core.mail.backends.smtp.EmailBackend"
-EMAIL_FILE_PATH="/tmp/app-messages"
-ENABLE_EMAIL=false
-PREDEFINED_TOKEN='5018dd65-9abd-4a62-8896-80f34ac66150'
-# TODO review these vars
-#SNAPSHOTS_DIR=/path/to/TODO
-#EVIDENCES_DIR=/path/to/TODO

admin/email.py

@@ -1,69 +0,0 @@
-import logging
-
-from django.conf import settings
-from django.template import loader
-from django.core.mail import EmailMultiAlternatives
-from django.contrib.auth.tokens import default_token_generator
-from django.contrib.sites.shortcuts import get_current_site
-from django.utils.encoding import force_bytes
-from django.utils.http import urlsafe_base64_encode
-
-
-logger = logging.getLogger(__name__)
-
-
-class NotifyActivateUserByEmail:
-    subject_template_name = 'activate_user_subject.txt'
-    email_template_name = 'activate_user_email.txt'
-    html_email_template_name = 'activate_user_email.html'
-
-    def get_email_context(self, user, token):
-        """
-        Define a new context with a token for put in a email
-        when send a email for add a new password  
-        """
-        protocol = 'https' if self.request.is_secure() else 'http'
-        current_site = get_current_site(self.request)
-        site_name = current_site.name
-        domain = current_site.domain
-        if not token:
-            token = default_token_generator.make_token(user)
-
-        context = {
-            'email': user.email,
-            'domain': domain,
-            'site_name': site_name,
-            'uid': urlsafe_base64_encode(force_bytes(user.pk)),
-            'user': user,
-            'token': token,
-            'protocol': protocol,
-        }
-        return context
-
-    def send_email(self, user, token=None):
-        """
-        Send a email when a user is activated.
-        """
-        context = self.get_email_context(user, token)
-        subject = loader.render_to_string(self.subject_template_name, context)
-        # Email subject *must not* contain newlines
-        subject = ''.join(subject.splitlines())
-        body = loader.render_to_string(self.email_template_name, context)
-        from_email = settings.DEFAULT_FROM_EMAIL
-        to_email = user.email
-
-        email_message = EmailMultiAlternatives(
-            subject, body, from_email, [to_email])
-        html_email = loader.render_to_string(self.html_email_template_name, context)
-        email_message.attach_alternative(html_email, 'text/html')
-        try:
-            if settings.ENABLE_EMAIL:
-                email_message.send()
-                return
-
-            logger.warning(to_email)
-            logger.warning(body)
-
-        except Exception as err:
-            logger.error(err)
-            return

admin/views.py

@@ -1,4 +1,3 @@
-from smtplib import SMTPException
 from django.urls import reverse_lazy
 from django.shortcuts import get_object_or_404
 from django.utils.translation import gettext_lazy as _
@@ -10,7 +9,6 @@ from django.views.generic.edit import (
 )
 from dashboard.mixins import DashboardView, Http403
 from user.models import User, Institution
-from admin.email import NotifyActivateUserByEmail
 
 
 class AdminView(DashboardView):
@@ -44,7 +42,7 @@ class UsersView(AdminView, TemplateView):
         return context
 
 
-class CreateUserView(AdminView, NotifyActivateUserByEmail, CreateView):
+class CreateUserView(AdminView, CreateView):
     template_name = "user.html"
     title = _("User")
     breadcrumb = _("admin / User") + " /"
@@ -60,12 +58,6 @@ class CreateUserView(AdminView, NotifyActivateUserByEmail, CreateView):
         form.instance.institution = self.request.user.institution
         form.instance.set_password(form.instance.password)
         response = super().form_valid(form)
-
-        try:
-            self.send_email(form.instance)
-        except SMTPException as e:
-            messages.error(self.request, e)
-
         return response
 
 

api/views.py

@@ -1,27 +1,32 @@
 import json
 
-from uuid import uuid4
-
 from django.urls import reverse_lazy
-from django.http import JsonResponse
 from django.shortcuts import get_object_or_404, redirect
 from django.utils.translation import gettext_lazy as _
 from django.views.decorators.csrf import csrf_exempt
+from django.core.exceptions import ValidationError
 from django_tables2 import SingleTableView
+from django.views.generic.base import View
 from django.views.generic.edit import (
     CreateView,
     DeleteView,
     UpdateView,
 )
+from django.http import JsonResponse
+from uuid import uuid4
 
-from utils.save_snapshots import move_json, save_in_disk
 from dashboard.mixins import DashboardView
 from evidence.models import Annotation
 from evidence.parse import Build
+from user.models import User
 from api.models import Token
 from api.tables import TokensTable
 
 
+def save_in_disk(data, user):
+    pass
+
+
 @csrf_exempt
 def NewSnapshot(request):
     # Accept only posts
@@ -54,42 +59,19 @@ def NewSnapshot(request):
     ).first()
 
     if exist_annotation:
-        txt = "error: the snapshot {} exist".format(data['uuid'])
-        return JsonResponse({'status': txt}, status=500)
+        raise ValidationError("error: the snapshot {} exist".format(data['uuid']))
 
     # Process snapshot
-    path_name = save_in_disk(data, tk.owner.institution.name)
+    # save_in_disk(data, tk.user)
 
     try:
         Build(data, tk.owner)
-    except Exception as err:
-        return JsonResponse({'status': f"fail: {err}"}, status=500)
+    except Exception:
+        return JsonResponse({'status': 'fail'}, status=200)
 
-    annotation = Annotation.objects.filter(
-        uuid=data['uuid'],
-        type=Annotation.Type.SYSTEM,
-        # TODO this is hardcoded, it should select the user preferred algorithm
-        key="hidalgo1",
-        owner=tk.owner.institution
-    ).first()
+    return JsonResponse({'status': 'success'}, status=200)
 
 
-    if not annotation:
-        return JsonResponse({'status': 'fail'}, status=500)
-
-    url_args = reverse_lazy("device:details", args=(annotation.value,))
-    url = request.build_absolute_uri(url_args)
-
-    response = {
-        "status": "success",
-        "dhid": annotation.value[:6].upper(),
-        "url": url,
-        # TODO replace with public_url when available
-        "public_url": url
-    }
-    move_json(path_name, tk.owner.institution.name)
-
-    return JsonResponse(response, status=200)
 
 
 class TokenView(DashboardView, SingleTableView):

dashboard/mixins.py

@@ -1,5 +1,4 @@
 from django.urls import resolve
-from django.conf import settings
 from django.shortcuts import get_object_or_404, redirect, Http404
 from django.utils.translation import gettext_lazy as _
 from django.core.exceptions import PermissionDenied
@@ -33,7 +32,6 @@ class DashboardView(LoginRequiredMixin):
     def get_context_data(self, **kwargs):
         context = super().get_context_data(**kwargs)
         context.update({
-            "commit_id": settings.COMMIT,
             'title': self.title,
             'subtitle': self.subtitle,
             'breadcrumb': self.breadcrumb,

dashboard/templates/unassigned_devices.html

@@ -15,7 +15,7 @@
       {% trans 'Documents' %}
     </a>
     {% endif %}
-    <a href="{# url 'dashboard:exports' object.id #}" type="button" class="btn btn-green-admin">
+    <a href="{# url 'idhub:admin_people_activate' object.id #}" type="button" class="btn btn-green-admin">
       <i class="bi bi-reply"></i>
       {% trans 'Exports' %}
     </a>

dashboard/views.py

@@ -69,17 +69,23 @@ class SearchView(InventaryMixin):
         if not matches.size():
             return self.search_hids(query, offset, limit)
 
-        devices = []
+        annotations = []
         for x in matches:
-            devices.append(self.get_annotations(x))
+            annotations.extend(self.get_annotations(x))
 
+        devices = [Device(id=x) for x in set(annotations)]
         count = matches.size()
         return devices, count
 
     def get_annotations(self, xp):
         snap = xp.document.get_data()
         uuid = json.loads(snap).get('uuid')
-        return Device.get_annotation_from_uuid(uuid, self.request.user.institution)
+
+        return Annotation.objects.filter(
+            type=Annotation.Type.SYSTEM,
+            owner=self.request.user.institution,
+            uuid=uuid
+        ).values_list("value", flat=True).distinct()
 
     def search_hids(self, query, offset, limit):
         qry = Q()

device/forms.py

@@ -1,6 +1,5 @@
 from django import forms
 from utils.device import create_annotation, create_doc, create_index
-from utils.save_snapshots import move_json, save_in_disk
 
 
 DEVICE_TYPES = [
@@ -57,11 +56,8 @@ class BaseDeviceFormSet(forms.BaseFormSet):
         if not commit:
             return doc
 
-        path_name = save_in_disk(doc, self.user.institution.name, place="placeholder")
         create_index(doc, self.user)
         create_annotation(doc, user, commit=commit)
-        move_json(path_name, self.user.institution.name, place="placeholder")
-        
         return doc
 
 

device/models.py

@@ -27,7 +27,7 @@ class Device:
         # the id is the chid of the device
         self.id = kwargs["id"]
         self.pk = self.id
-        self.shortid = self.pk[:6].upper()
+        self.shortid = self.pk[:6]
         self.algorithm = None
         self.owner = None
         self.annotations =  []
@@ -90,11 +90,9 @@ class Device:
     def get_hids(self):
         annotations = self.get_annotations()
 
-        algos = list(ALGOS.keys())
-        algos.append('CUSTOM_ID')
         self.hids = list(set(annotations.filter(
             type=Annotation.Type.SYSTEM,
-            key__in=algos,
+            key__in=ALGOS.keys(),
         ).values_list("value", flat=True)))
 
     def get_evidences(self):
@@ -120,32 +118,9 @@ class Device:
     def get_unassigned(cls, institution, offset=0, limit=None):
 
         sql = """
-            WITH RankedAnnotations AS (
-                SELECT
-                    t1.value,
-                    t1.key,
-                    ROW_NUMBER() OVER (
-                        PARTITION BY t1.uuid
-                        ORDER BY
-                            CASE
-                                WHEN t1.key = 'CUSTOM_ID' THEN 1
-                                WHEN t1.key = 'hidalgo1' THEN 2
-                                ELSE 3
-                            END,
-                            t1.created DESC
-                    ) AS row_num
-                FROM evidence_annotation AS t1
-                LEFT JOIN lot_devicelot AS t2 ON t1.value = t2.device_id
-                WHERE t2.device_id IS NULL
-                  AND t1.owner_id = {institution}
-                  AND t1.type = {type}
-            )
-            SELECT DISTINCT
-                value
-            FROM
-                RankedAnnotations
-            WHERE
-                row_num = 1
+              SELECT DISTINCT t1.value from evidence_annotation as t1
+                left join lot_devicelot as t2 on t1.value = t2.device_id
+              where t2.device_id is null and owner_id=={institution} and type=={type}
         """.format(
             institution=institution.id,
             type=Annotation.Type.SYSTEM,
@@ -169,83 +144,18 @@ class Device:
     def get_unassigned_count(cls, institution):
 
         sql = """
-            WITH RankedAnnotations AS (
-                SELECT
-                    t1.value,
-                    t1.key,
-                    ROW_NUMBER() OVER (
-                        PARTITION BY t1.uuid
-                        ORDER BY
-                            CASE
-                                WHEN t1.key = 'CUSTOM_ID' THEN 1
-                                WHEN t1.key = 'hidalgo1' THEN 2
-                                ELSE 3
-                            END,
-                            t1.created DESC
-                    ) AS row_num
-                FROM evidence_annotation AS t1
-                LEFT JOIN lot_devicelot AS t2 ON t1.value = t2.device_id
-                WHERE t2.device_id IS NULL
-                  AND t1.owner_id = {institution}
-                  AND t1.type = {type}
-            )
-            SELECT
-                COUNT(DISTINCT value)
-            FROM
-                RankedAnnotations
-            WHERE
-                row_num = 1
+              SELECT count(DISTINCT t1.value) from evidence_annotation as t1
+                left join lot_devicelot as t2 on t1.value = t2.device_id
+              where t2.device_id is null and owner_id=={institution} and type=={type};
         """.format(
             institution=institution.id,
             type=Annotation.Type.SYSTEM,
         )
+
         with connection.cursor() as cursor:
             cursor.execute(sql)
             return cursor.fetchall()[0][0]
 
-    @classmethod
-    def get_annotation_from_uuid(cls, uuid, institution):
-        sql = """
-            WITH RankedAnnotations AS (
-                SELECT
-                    t1.value,
-                    t1.key,
-                    ROW_NUMBER() OVER (
-                        PARTITION BY t1.uuid
-                        ORDER BY
-                            CASE
-                                WHEN t1.key = 'CUSTOM_ID' THEN 1
-                                WHEN t1.key = 'hidalgo1' THEN 2
-                                ELSE 3
-                            END,
-                            t1.created DESC
-                    ) AS row_num
-                FROM evidence_annotation AS t1
-                LEFT JOIN lot_devicelot AS t2 ON t1.value = t2.device_id
-                WHERE t2.device_id IS NULL
-                  AND t1.owner_id = {institution}
-                  AND t1.type = {type}
-                  AND t1.uuid = '{uuid}'
-            )
-            SELECT DISTINCT
-                value
-            FROM
-                RankedAnnotations
-            WHERE
-                row_num = 1;
-        """.format(
-            uuid=uuid.replace("-", ""),
-            institution=institution.id,
-            type=Annotation.Type.SYSTEM,
-        )
-
-        annotations = []
-        with connection.cursor() as cursor:
-            cursor.execute(sql)
-            annotations = cursor.fetchall()
-
-        return cls(id=annotations[0][0])
-
     @property
     def is_websnapshot(self):
         if not self.last_evidence:

device/templates/details.html

@@ -12,25 +12,25 @@
   <div class="col">
     <ul class="nav nav-tabs nav-tabs-bordered">
       <li class="nav-items">
-        <a href="#details" class="nav-link active" data-bs-toggle="tab" data-bs-target="#details">General details</a>
+        <button class="nav-link active" data-bs-toggle="tab" data-bs-target="#details">General details</button>
       </li>
       <li class="nav-items">
-        <a href="#annotations" class="nav-link" data-bs-toggle="tab" data-bs-target="#annotations">User annotations</a>
+        <button class="nav-link" data-bs-toggle="tab" data-bs-target="#annotations">User annotations</button>
       </li>
       <li class="nav-items">
-        <a href="#documents" class="nav-link" data-bs-toggle="tab" data-bs-target="#documents">Documents</a>
+        <button class="nav-link" data-bs-toggle="tab" data-bs-target="#documents">Documents</button>
       </li>
       <li class="nav-items">
-        <a href="#lots" class="nav-link" data-bs-toggle="tab" data-bs-target="#lots">Lots</a>
+        <button class="nav-link" data-bs-toggle="tab" data-bs-target="#lots">Lots</button>
       </li>
       <li class="nav-items">
-        <a href="#components" class="nav-link" data-bs-toggle="tab" data-bs-target="#components">Components</a>
+        <button class="nav-link" data-bs-toggle="tab" data-bs-target="#components">Components</button>
       </li>
       <li class="nav-items">
-        <a href="#evidences" class="nav-link" data-bs-toggle="tab" data-bs-target="#evidences">Evidences</a>
+        <button class="nav-link" data-bs-toggle="tab" data-bs-target="#evidences">Evidences</button>
       </li>
       <li class="nav-items">
-        <a href="#web" class="nav-link" href="">Web</a>
+        <a class="nav-link" href="">Web</a>
       </li>
     </ul>
   </div>
@@ -214,25 +214,3 @@
   </div>
 </div>
 {% endblock %}
-
-{% block extrascript %}
-<script>
-  document.addEventListener("DOMContentLoaded", function() {
-    // Obtener el hash de la URL (ejemplo: #components)
-    const hash = window.location.hash;
-
-    // Verificar si hay un hash en la URL
-    if (hash) {
-      // Buscar el botón o enlace que corresponde al hash y activarlo
-      const tabTrigger = document.querySelector(`[data-bs-target="${hash}"]`);
-      
-      if (tabTrigger) {
-        // Crear una instancia de tab de Bootstrap para activar el tab
-        const tab = new bootstrap.Tab(tabTrigger);
-        tab.show();
-      }
-    }
-  });
-</script>
-{% endblock %}
-

dhub/settings.py

@@ -10,7 +10,6 @@ For the full list of settings and their values, see
 https://docs.djangoproject.com/en/5.0/ref/settings/
 """
 
-import os
 import xapian
 
 from pathlib import Path
@@ -36,35 +35,10 @@ assert DOMAIN not in [None, ''], "DOMAIN var is MANDATORY"
 print("DOMAIN: " + DOMAIN)
 
 ALLOWED_HOSTS = config('ALLOWED_HOSTS', default=DOMAIN, cast=Csv())
-assert DOMAIN in ALLOWED_HOSTS, f"DOMAIN {DOMAIN} is not in ALLOWED_HOSTS {ALLOWED_HOSTS}"
+assert DOMAIN in ALLOWED_HOSTS, "DOMAIN is not ALLOWED_HOST"
 
 CSRF_TRUSTED_ORIGINS = config('CSRF_TRUSTED_ORIGINS', default=f'https://{DOMAIN}', cast=Csv())
 
-
-INITIAL_ADMIN_EMAIL = config("INITIAL_ADMIN_EMAIL", default='admin@example.org')
-INITIAL_ADMIN_PASSWORD = config("INITIAL_ADMIN_PASSWORD", default='1234')
-
-DEFAULT_FROM_EMAIL = config(
-    'DEFAULT_FROM_EMAIL', default='webmaster@localhost')
-
-EMAIL_HOST = config('EMAIL_HOST', default='localhost')
-
-EMAIL_HOST_USER = config('EMAIL_HOST_USER', default='')
-
-EMAIL_HOST_PASSWORD = config('EMAIL_HOST_PASSWORD', default='')
-
-EMAIL_PORT = config('EMAIL_PORT', default=25, cast=int)
-
-EMAIL_USE_TLS = config('EMAIL_USE_TLS', default=False, cast=bool)
-
-EMAIL_BACKEND = config('EMAIL_BACKEND', default='django.core.mail.backends.smtp.EmailBackend')
-
-EMAIL_FILE_PATH = config('EMAIL_FILE_PATH', default='/tmp/app-messages')
-
-ENABLE_EMAIL = config("ENABLE_EMAIL", default=True, cast=bool)
-
-EVIDENCES_DIR = config("EVIDENCES_DIR", default=os.path.join(BASE_DIR, "db"))
-
 # Application definition
 
 INSTALLED_APPS = [
@@ -158,20 +132,12 @@ AUTH_PASSWORD_VALIDATORS = [
 
 LANGUAGE_CODE = "en-us"
 
-TIME_ZONE = config("TIME_ZONE", default="UTC")
+TIME_ZONE = "UTC"
 
 USE_I18N = True
 
-USE_TZ = False
+USE_TZ = True
 
-if TIME_ZONE == "UTC":
-    USE_TZ = True
-
-USE_L10N = True
-LANGUAGES = [
-    ('es', 'Spanish'),
-    ('en', 'English'),
-]
 
 # Static files (CSS, JavaScript, Images)
 # https://docs.djangoproject.com/en/5.0/howto/static-files/
@@ -216,4 +182,3 @@ LOGGING = {
 
 SNAPSHOT_PATH="/tmp/"
 DATA_UPLOAD_MAX_NUMBER_FILES = 1000
-COMMIT = config('COMMIT', default='')

docker-compose.yml

@@ -6,9 +6,7 @@ services:
     environment:
       - DEBUG=true
       - DOMAIN=${DOMAIN:-localhost}
-      - ALLOWED_HOSTS=${ALLOWED_HOSTS:-$DOMAIN}
-      - DEMO=${DEMO:-false}
-      - PREDEFINED_TOKEN=${PREDEFINED_TOKEN:-}
+      - DEMO=${DEMO:-n}
     volumes:
       - .:/opt/devicehub-django
     ports:

docker-reset.sh

@@ -9,13 +9,11 @@ set -u
 set -x
 
 main() {
-        cd "$(dirname "${0}")"
-
         if [ "${DETACH:-}" ]; then
                 detach_arg='-d'
         fi
         # remove old database
-        sudo rm -vfr ./db/*
+        sudo rm -vf db/*
         docker compose down -v
         docker compose build
         docker compose up ${detach_arg:-}

docker/devicehub-django.entrypoint.sh

@@ -12,14 +12,6 @@ check_app_is_there() {
 }
 
 deploy() {
-        # TODO this is weird, find better workaround
-        git config --global --add safe.directory /opt/devicehub-django
-        export COMMIT=$(git log --format="%H %ad" --date=iso -n 1)
-
-        if [ "${DEBUG:-}" = 'true' ]; then
-                ./manage.py print_settings
-        fi
-
         # detect if existing deployment (TODO only works with sqlite)
         if [ -f "${program_dir}/db/db.sqlite3" ]; then
                 echo "INFO: detected EXISTING deployment"
@@ -32,13 +24,11 @@ deploy() {
                 INIT_ORG="${INIT_ORG:-example-org}"
                 INIT_USER="${INIT_USER:-user@example.org}"
                 INIT_PASSWD="${INIT_PASSWD:-1234}"
-                ADMIN='True'
-                PREDEFINED_TOKEN="${PREDEFINED_TOKEN:-}"
                 ./manage.py add_institution "${INIT_ORG}"
                 # TODO: one error on add_user, and you don't add user anymore
-                ./manage.py add_user "${INIT_ORG}" "${INIT_USER}" "${INIT_PASSWD}" "${ADMIN}" "${PREDEFINED_TOKEN}"
+                ./manage.py add_user "${INIT_ORG}" "${INIT_USER}" "${INIT_PASSWD}"
 
-                if [ "${DEMO:-}" = 'true' ]; then
+                if [ "${DEMO:-}" ]; then
                         ./manage.py up_snapshots example/snapshots/ "${INIT_USER}"
                 fi
         fi
@@ -46,7 +36,7 @@ deploy() {
 
 runserver() {
         PORT="${PORT:-8000}"
-        if [ "${DEBUG:-}" = 'true' ]; then
+        if [ "${DEBUG:-}" ]; then
                 ./manage.py runserver 0.0.0.0:${PORT}
         else
                 # TODO

evidence/forms.py

@@ -9,7 +9,6 @@ from utils.forms import MultipleFileField
 from device.models import Device
 from evidence.parse import Build
 from evidence.models import Annotation
-from utils.save_snapshots import move_json, save_in_disk
 
 
 class UploadForm(forms.Form):
@@ -49,9 +48,7 @@ class UploadForm(forms.Form):
             return
 
         for ev in self.evidences:
-            path_name = save_in_disk(ev[1], user.institution.name)
             Build(ev[1], user)
-            move_json(path_name, user.institution.name)
 
 
 class UserTagForm(forms.Form):
@@ -154,11 +151,8 @@ class ImportForm(forms.Form):
 
         if commit:
             for doc, cred in table:
-                path_name = save_in_disk(doc, self.user.institution.name, place="placeholder")
-
               cred.save()
               create_index(doc, self.user)
-                move_json(path_name, self.user.institution.name, place="placeholder")
             return table
 
         return

evidence/models.py

@@ -67,7 +67,7 @@ class Evidence:
         for xa in matches:
             self.doc = json.loads(xa.document.get_data())
 
-        if self.doc.get("software") == "workbench-script":
+        if self.doc.get("software") == "EreuseWorkbench":
             dmidecode_raw = self.doc["data"]["dmidecode"]
             self.dmi = DMIParse(dmidecode_raw)
 
@@ -80,7 +80,7 @@ class Evidence:
             self.created = self.annotations.last().created
 
     def get_components(self):
-        if self.doc.get("software") != "workbench-script":
+        if self.doc.get("software") != "EreuseWorkbench":
             return self.doc.get('components', [])
         self.set_components()
         return self.components
@@ -92,7 +92,7 @@ class Evidence:
                 return ""
             return list(self.doc.get('kv').values())[0]
 
-        if self.doc.get("software") != "workbench-script":
+        if self.doc.get("software") != "EreuseWorkbench":
             return self.doc['device']['manufacturer']
 
         return self.dmi.manufacturer().strip()
@@ -104,13 +104,13 @@ class Evidence:
                 return ""
             return list(self.doc.get('kv').values())[1]
 
-        if self.doc.get("software") != "workbench-script":
+        if self.doc.get("software") != "EreuseWorkbench":
             return self.doc['device']['model']
 
         return self.dmi.model().strip()
 
     def get_chassis(self):
-        if self.doc.get("software") != "workbench-script":
+        if self.doc.get("software") != "EreuseWorkbench":
             return self.doc['device']['model']
 
         chassis = self.dmi.get("Chassis")[0].get("Type", '_virtual')
@@ -126,8 +126,7 @@ class Evidence:
         return Annotation.objects.filter(
             owner=user.institution,
             type=Annotation.Type.SYSTEM,
-            key="hidalgo1",
-        ).order_by("-created").values_list("uuid", "created").distinct()
+        ).order_by("-created").values_list("uuid", flat=True).distinct()
 
     def set_components(self):
         snapshot = ParseSnapshot(self.doc).snapshot_json

evidence/parse.py

@@ -5,8 +5,6 @@ import hashlib
 
 from datetime import datetime
 from dmidecode import DMIParse
-from json_repair import repair_json
-
 from evidence.models import Annotation
 from evidence.xapian import index
 from utils.constants import ALGOS, CHASSIS_DH
@@ -22,12 +20,7 @@ def get_network_cards(child, nets):
 def get_mac(lshw):
     nets = []
     try:
-        hw = json.loads(lshw)
-    except json.decoder.JSONDecodeError:
-        hw = json.loads(repair_json(lshw))
-        
-    try:
-        get_network_cards(hw, nets)
+        get_network_cards(json.loads(lshw), nets)
     except Exception as ss:
         print("WARNING!! {}".format(ss))
         return
@@ -64,7 +57,7 @@ class Build:
         }
 
     def get_hid_14(self):
-        if self.json.get("software") == "workbench-script":
+        if self.json.get("software") == "EreuseWorkbench":
             hid = self.get_hid(self.json)
         else:
             device = self.json['device']
@@ -120,8 +113,7 @@ class Build:
         # mac = get_mac2(hwinfo_raw) or ""
         mac = get_mac(lshw) or ""
         if not mac:
-            print(f"WARNING: Could not retrieve MAC address in snapshot {snapshot['uuid']}" )
-            # TODO generate system annotation for that snapshot
+            print("WARNING!! No there are MAC address")
         else:
             print(f"{manufacturer}{model}{chassis}{serial_number}{sku}{mac}")
 

evidence/parse_details.py

@@ -3,8 +3,6 @@ import numpy as np
 
 from datetime import datetime
 from dmidecode import DMIParse
-from json_repair import repair_json
-
 from utils.constants import CHASSIS_DH, DATASTORAGEINTERFACE
 
 
@@ -162,7 +160,6 @@ class ParseSnapshot:
                 continue
             model = sm.get('model_name')
             manufacturer = None
-            hours = sm.get("power_on_time", {}).get("hours", 0)
             if model and len(model.split(" ")) > 1:
                 mm = model.split(" ")
                 model = mm[-1]
@@ -178,7 +175,6 @@ class ParseSnapshot:
                     "size": self.get_data_storage_size(sm),
                     "variant": sm.get("firmware_version"),
                     "interface": self.get_data_storage_interface(sm),
-                    "hours": hours,
                 }
             )
 
@@ -482,11 +478,7 @@ class ParseSnapshot:
     def loads(self, x):
         if isinstance(x, str):
             try:
-                try:
-                    hw = json.loads(lshw)
-                except json.decoder.JSONDecodeError:
-                    hw = json.loads(repair_json(lshw))
-                return hw
+                return json.loads(x)
             except Exception as ss:
                 print("WARNING!! {}".format(ss))
                 return {}

evidence/templates/evidences.html

@@ -14,13 +14,10 @@
     {% for ev in evidences %}
     <tr>
       <td>
-        <a href="{% url 'evidence:details' ev.0 %}">{{ ev.0 }}</a>
+        <a href="{% url 'evidence:details' ev %}">{{ ev }}</a>
       </td>
       <td>
-	<small class="text-muted">{{ ev.1 }}</small>
-      </td>
-      <td>
-        <a href="{# url 'evidence:delete' ev.0 #}"><i class="bi bi-trash text-danger"></i></a>
+        <a href="{# url 'evidence:delete' ev #}"><i class="bi bi-trash text-danger"></i></a>
       </td>
     </tr>
     {% endfor %}

login/templates/activate_user_email.html

@@ -1,31 +0,0 @@
-{% load i18n %}{% autoescape off %}
-{% trans "DeviceHub" as site %}
-<p>
-  {% blocktrans %}You're receiving this email because your user account at {{site}} has been activated.{% endblocktrans %}
-</p>
-
-<p>
-{% trans "Your username is:" %} {{ user.username }}
-</p>
-
-<p>
-{% trans "Please go to the following page and choose a password:" %}
-</p>
-
-<p>
-{% block reset_link %}
-<a href="{{ protocol }}://{{ domain }}{% url 'login:password_reset_confirm' uidb64=uid token=token %}">
-{{ protocol }}://{{ domain }}{% url 'login:password_reset_confirm' uidb64=uid token=token %}
-</a>
-{% endblock %}
-</p>
-
-<p>
-{% trans "Thanks for using our site!" %}
-</p>
-
-<p>
-{% blocktrans %}The {{site}} team{% endblocktrans %}
-</p>
-
-{% endautoescape %}

login/templates/activate_user_email.txt

@@ -1,19 +0,0 @@
-{% load i18n %}{% autoescape off %}
-
-{% trans "DeviceHub" as site %}
-
-{% blocktrans %}You're receiving this email because your user account at {{site}} has been activated.{% endblocktrans %}
-
-{% trans "Your username is:" %} {{ user.username }}
-
-{% trans "Please go to the following page and choose a password:" %}
-{% block reset_link %}
-{{ protocol }}://{{ domain }}{% url 'login:password_reset_confirm' uidb64=uid token=token %}
-{% endblock %}
-
-
-{% trans "Thanks for using our site!" %}
-
-{% blocktrans %}The {{site}} team{% endblocktrans %}
-
-{% endautoescape %}

login/templates/activate_user_subject.txt

@@ -1,4 +0,0 @@
-{% load i18n %}{% autoescape off %}
-{% trans "IdHub" as site %}
-{% blocktrans %}User activation on {{site}}{% endblocktrans %}
-{% endautoescape %}

login/templates/login.html

@@ -42,5 +42,4 @@
 <div id="login-footer" class="mt-3">
     <a href="{% url 'login:password_reset' %}" data-toggle="modal" data-target="#forgotPasswordModal">{% trans "Forgot your password? Click here to recover" %}</a>
 </div>
-
 {% endblock %}

login/templates/password_reset_email.html

@@ -9,8 +9,8 @@
 
 <p>
 {% block reset_link %}
-<a href="{{ protocol }}://{{ domain }}{% url 'login:password_reset_confirm' uidb64=uid token=token %}">
-{{ protocol }}://{{ domain }}{% url 'login:password_reset_confirm' uidb64=uid token=token %}
+<a href="{{ protocol }}://{{ domain }}{% url 'idhub:password_reset_confirm' uidb64=uid token=token %}">
+{{ protocol }}://{{ domain }}{% url 'idhub:password_reset_confirm' uidb64=uid token=token %}
 </a>
 {% endblock %}
 </p>

login/templates/password_reset_email.txt

@@ -3,7 +3,7 @@
 
 {% trans "Please go to the following page and choose a new password:" %}
 {% block reset_link %}
-{{ protocol }}://{{ domain }}{% url 'login:password_reset_confirm' uidb64=uid token=token %}
+{{ protocol }}://{{ domain }}{% url 'idhub:password_reset_confirm' uidb64=uid token=token %}
 {% endblock %}
 {% trans "Your username, in case you've forgotten:" %} {{ user.username }}
 

login/views.py

@@ -1,6 +1,5 @@
 import logging
 
-from django.conf import settings
 from django.urls import reverse_lazy
 from django.contrib.auth import views as auth_views
 from django.contrib.auth import login as auth_login
@@ -18,7 +17,7 @@ class LoginView(auth_views.LoginView):
     extra_context = {
         'title': _('Login'),
         'success_url': reverse_lazy('dashboard:unassigned_devices'),
-        'commit_id': settings.COMMIT,
+        # 'commit_id': settings.COMMIT, 
     }
 
     def get(self, request, *args, **kwargs):
@@ -67,8 +66,7 @@ class PasswordResetView(auth_views.PasswordResetView):
 
     def form_valid(self, form):
         try:
-            response = super().form_valid(form)
-            return response
+            return super().form_valid(form)
         except Exception as err:
             logger.error(err)
         return HttpResponseRedirect(self.success_url)

requirements.txt

@@ -10,4 +10,4 @@ pandas==2.2.2
 xlrd==2.0.1
 odfpy==1.4.1
 pytz==2024.2
-json-repair==0.30.0
+

user/management/commands/add_user.py

@@ -17,17 +17,15 @@ class Command(BaseCommand):
         parser.add_argument('email', type=str, help='email')
         parser.add_argument('password', type=str, help='password')
         parser.add_argument('is_admin', nargs='?', default=False, type=str, help='is admin')
-        parser.add_argument('predefined_token', nargs='?', default='', type=str, help='predefined token')
 
     def handle(self, *args, **kwargs):
         email = kwargs['email']
         password = kwargs['password']
         is_admin = kwargs['is_admin']
-        predefined_token = kwargs['predefined_token']
         institution = Institution.objects.get(name=kwargs['institution'])
-        self.create_user(institution, email, password, is_admin, predefined_token)
+        self.create_user(institution, email, password, is_admin)
 
-    def create_user(self, institution, email, password, is_admin, predefined_token):
+    def create_user(self, institution, email, password, is_admin):
         self.u = User.objects.create(
             institution=institution,
             email=email,
@@ -36,10 +34,6 @@ class Command(BaseCommand):
         )
         self.u.set_password(password)
         self.u.save()
-        if predefined_token:
-            token = predefined_token
-        else:
         token = uuid4()
-
         Token.objects.create(token=token, owner=self.u)
         print(f"TOKEN: {token}")

utils/save_snapshots.py

@@ -1,43 +0,0 @@
-import os
-import json
-import shutil
-
-from datetime import datetime
-from django.conf import settings
-
-
-def move_json(path_name, user, place="snapshots"):
-    if place != "snapshots":
-        place = "placeholders"
-
-    tmp_snapshots = settings.EVIDENCES_DIR
-    path_dir = os.path.join(tmp_snapshots, user, place)
-
-    if os.path.isfile(path_name):
-        shutil.copy(path_name, path_dir)
-        os.remove(path_name)
-
-
-def save_in_disk(data, user, place="snapshots"):
-    uuid = data.get('uuid', '')
-    now = datetime.now()
-    year = now.year
-    month = now.month
-    day = now.day
-    hour = now.hour
-    minutes = now.minute
-    tmp_snapshots = settings.EVIDENCES_DIR
-    if place != "snapshots":
-        place = "placeholders"
-
-    name_file = f"{year}-{month}-{day}-{hour}-{minutes}_{uuid}.json"
-    path_dir = os.path.join(tmp_snapshots, user, place, "errors")
-    path_name = os.path.join(path_dir, name_file)
-
-    if not os.path.isdir(path_dir):
-        os.system(f'mkdir -p {path_dir}')
-
-    with open(path_name, 'w') as snapshot_file:
-        snapshot_file.write(json.dumps(data))
-
-    return path_name