From e7519b5b33ee5e2af08093fed90dab5119a13771 Mon Sep 17 00:00:00 2001
From: lgromero <lgromero@qindel.com>
Date: Tue, 1 Oct 2024 16:13:57 +0200
Subject: [PATCH] refs #799 fix a error in the comment kea ctrl authentication
 and a typo in the swagger doc

---
 installer/ogdhcp_installer.sh                | 64 ++++++++++----------
 src/DhcpBundle/Controller/DhcpController.php |  2 +-
 2 files changed, 33 insertions(+), 33 deletions(-)

diff --git a/installer/ogdhcp_installer.sh b/installer/ogdhcp_installer.sh
index 80f6cba..4151272 100755
--- a/installer/ogdhcp_installer.sh
+++ b/installer/ogdhcp_installer.sh
@@ -295,11 +295,14 @@ get_first_network_interface_with_traffic() {
 }
 comment_auth_kea() {
     KEA_CTRL_AGENT_CONF="/etc/kea/kea-ctrl-agent.conf"
+    
+    # Verificar si el bloque de "authentication" ya está comentado
     if grep -q '^[^#]*"authentication": {' "$KEA_CTRL_AGENT_CONF"; then
         echo "Comentando el bloque de autenticación en $KEA_CTRL_AGENT_CONF..."
-        sed -i '/"authentication": {/,/]/ {
-            s/^ *//;   # Elimina espacios en blanco al principio de la línea
-            s/^\([^#]\)/#\1/  # Comenta solo las líneas no comentadas
+
+        # Comentar solo el bloque de authentication desde la apertura hasta la línea con '},'
+        sed -i '/"authentication": {/,/^[[:space:]]*},/ {
+            s/^\([[:space:]]*\)\([^#]\)/\1#\2/
         }' "$KEA_CTRL_AGENT_CONF"
 
         echo "Bloque de autenticación comentado correctamente."
@@ -307,10 +310,10 @@ comment_auth_kea() {
         echo "El bloque de autenticación ya está comentado."
     fi
 
-    # Verificar si las líneas fueron comentadas usando una expresión regular mejorada
-    # que permite cualquier cantidad de espacios entre # y "authentication"
+    # Verificar si el bloque fue comentado correctamente
     if grep -q '^#\s*"authentication": {' "$KEA_CTRL_AGENT_CONF"; then
         echo "Confirmación: Bloque de autenticación comentado correctamente."
+
         # Reiniciar el servicio de Kea Control Agent para aplicar los cambios
         echo "Reiniciando el agente de Kea Control Agent..."
         sudo systemctl restart kea-ctrl-agent.service
@@ -335,37 +338,34 @@ get_php_fpm_version() {
     php -v | grep -oP "PHP \K\d+\.\d+"
 }
 add_write_permission_apparmor() {
-# Archivo de perfil de AppArmor
-APPARMOR_FILE="/etc/apparmor.d/usr.sbin.kea-dhcp4"
+    APPARMOR_FILE="/etc/apparmor.d/usr.sbin.kea-dhcp4"
+    KEA_RULES="/etc/kea/ rw,
+    /etc/kea/** rw,"
+    KEA_CONFIG_FILE="/etc/kea/kea-dhcp4.conf"
 
-# Comprobar si ya existe la regla para /etc/kea/
-if ! grep -q "/etc/kea/ " "$APPARMOR_FILE"; then
-    echo "Añadiendo la regla para /etc/kea/..."
-    sudo sed -i '/#include <tunables\/global>/a \
-/etc/kea/ rw,' "$APPARMOR_FILE"
-else
-    echo "La regla para /etc/kea/ ya existe."
-    # Asegurarse de que tiene los permisos correctos
-    sudo sed -i 's|/etc/kea/.*|/etc/kea/ rw,|' "$APPARMOR_FILE"
-fi
+    # Comprobar si la línea "/etc/kea/ rw," ya existe
+    if ! grep -q "/etc/kea/ rw," "$APPARMOR_FILE"; then
+        echo "Añadiendo /etc/kea/ rw, al archivo $APPARMOR_FILE"
+        echo "/etc/kea/ rw," | sudo tee -a "$APPARMOR_FILE" > /dev/null
+    else
+        echo "La regla '/etc/kea/ rw,' ya existe."
+    fi
 
-# Comprobar si ya existe la regla para /etc/kea/**
-if ! grep -q "/etc/kea/\*\* " "$APPARMOR_FILE"; then
-    echo "Añadiendo la regla para /etc/kea/**..."
-    sudo sed -i '/#include <tunables\/global>/a \
-/etc/kea/** rw,' "$APPARMOR_FILE"
-else
-    echo "La regla para /etc/kea/** ya existe."
-    # Asegurarse de que tiene los permisos correctos
-    sudo sed -i 's|/etc/kea/\*\*.*|/etc/kea/** rw,|' "$APPARMOR_FILE"
-fi
+    # Comprobar si la línea "/etc/kea/** rw," ya existe
+    if ! grep -q "/etc/kea/** rw," "$APPARMOR_FILE"; then
+        echo "Añadiendo /etc/kea/** rw, al archivo $APPARMOR_FILE"
+        echo "/etc/kea/** rw," | sudo tee -a "$APPARMOR_FILE" > /dev/null
+    else
+        echo "La regla '/etc/kea/** rw,' ya existe."
+    fi
 
-# Recargar AppArmor para aplicar los cambios
-echo "Recargando AppArmor..."
-sudo apparmor_parser -r "$APPARMOR_FILE"
-sudo systemctl restart kea-ctrl-agent kea-dhcp4
+    # Establecer permisos y grupo en /etc/kea/kea-dhcp4.conf
+    echo "Estableciendo permisos de _kea:_kea en $KEA_CONFIG_FILE"
+    sudo chown _kea:_kea "$KEA_CONFIG_FILE"
 
-echo "Perfil de AppArmor modificado y servicios reiniciados."
+    # Recargar AppArmor
+    echo "Recargando AppArmor..."
+    sudo systemctl reload apparmor
 }
 
 # Función para configurar Nginx
diff --git a/src/DhcpBundle/Controller/DhcpController.php b/src/DhcpBundle/Controller/DhcpController.php
index 2495450..9d4af1a 100644
--- a/src/DhcpBundle/Controller/DhcpController.php
+++ b/src/DhcpBundle/Controller/DhcpController.php
@@ -180,7 +180,7 @@ class DhcpController
  *     @OA\Response(
  *         response=400,
  *         description="Error al obtener las subredes",
- *     )executeCurlCommand
+ *     )
  * )
  * @Route("/ogdhcp/v1/subnets", methods={"GET"})
  */