Merge branch 'master' into stable

(cherry picked from commit 3e0ffec07c)

frontend/src/components/InterfaceEditModal.vue

@@ -315,7 +315,6 @@ async function applyPeerDefaults() {
 
 async function del() {
   if (isDeleting.value) return
-  if (!confirm(t('modals.interface-edit.confirm-delete', {id: selectedInterface.value.Identifier}))) return
   isDeleting.value = true
   try {
     await interfaces.DeleteInterface(selectedInterface.value.Identifier)

frontend/src/components/PeerEditModal.vue

@@ -294,7 +294,6 @@ async function save() {
 
 async function del() {
   if (isDeleting.value) return
-  if (!confirm(t('modals.peer-edit.confirm-delete', {id: selectedPeer.value.Identifier}))) return
   isDeleting.value = true
   try {
     await peers.DeletePeer(selectedPeer.value.Identifier)

frontend/src/components/UserEditModal.vue

@@ -114,7 +114,6 @@ async function save() {
 
 async function del() {
   if (isDeleting.value) return
-  if (!confirm(t('modals.user-edit.confirm-delete', {id: selectedUser.value.Identifier}))) return
   isDeleting.value = true
   try {
     await users.DeleteUser(selectedUser.value.Identifier)

frontend/src/lang/translations/de.json

@@ -382,8 +382,7 @@
       "persist-local-changes": {
         "label": "Lokale Änderungen speichern"
       },
-      "sync-warning": "Um diesen synchronisierten Benutzer zu bearbeiten, aktivieren Sie die lokale Änderungsspeicherung. Andernfalls werden Ihre Änderungen bei der nächsten Synchronisierung überschrieben.",
-      "confirm-delete": "Benutzer '{id}' wirklich löschen?"
+      "sync-warning": "Um diesen synchronisierten Benutzer zu bearbeiten, aktivieren Sie die lokale Änderungsspeicherung. Andernfalls werden Ihre Änderungen bei der nächsten Synchronisierung überschrieben."
     },
     "interface-view": {
       "headline": "Konfiguration für Schnittstelle:"
@@ -504,8 +503,7 @@
           "placeholder": "Persistentes Keepalive (0 = Standard)"
         }
       },
-      "button-apply-defaults": "Peer-Standardeinstellungen anwenden",
-      "confirm-delete": "Interface '{id}' wirklich löschen?"
+      "button-apply-defaults": "Peer-Standardeinstellungen anwenden"
     },
     "peer-view": {
       "headline-peer": "Peer:",
@@ -627,8 +625,7 @@
       },
       "expires-at": {
         "label": "Ablaufdatum"
-      },
-      "confirm-delete": "Peer '{id}' wirklich löschen?"
+      }
     },
     "peer-multi-create": {
       "headline-peer": "Mehrere Peers erstellen",

frontend/src/lang/translations/en.json

@@ -271,16 +271,16 @@
     "headline-preshared-key": "New Preshared Key",
     "button-generate": "Generate",
     "private-key": {
-      "label": "Private Key",
-      "placeholder": "The private key"
+        "label": "Private Key",
+        "placeholder": "The private key"
     },
     "public-key": {
-      "label": "Public Key",
-      "placeholder": "The public key"
+        "label": "Public Key",
+        "placeholder": "The public key"
     },
     "preshared-key": {
-      "label": "Preshared Key",
-      "placeholder": "The pre-shared key"
+        "label": "Preshared Key",
+        "placeholder": "The pre-shared key"
     }
   },
   "calculator": {
@@ -289,18 +289,18 @@
     "headline-allowed-ip": "New Allowed IPs",
     "button-exclude-private": "Exclude Private IP Ranges",
     "allowed-ip": {
-      "label": "Allowed IPs",
-      "placeholder": "0.0.0.0/0, ::/0",
-      "empty": "Value cannot be empty"
+        "label": "Allowed IPs",
+        "placeholder": "0.0.0.0/0, ::/0",
+        "empty": "Value cannot be empty"
     },
     "dissallowed-ip": {
-      "label": "Disallowed IPs",
-      "placeholder": "10.0.0.0/8, 192.168.0.0/16",
-      "invalid": "Invalid address: {addr}"
+        "label": "Disallowed IPs",
+        "placeholder": "10.0.0.0/8, 192.168.0.0/16",
+        "invalid": "Invalid address: {addr}"
     },
     "new-allowed-ip": {
-      "label": "Allowed IPs",
-      "placeholder": ""
+        "label": "Allowed IPs",
+        "placeholder": ""
     }
   },
   "modals": {
@@ -382,8 +382,7 @@
       "persist-local-changes": {
         "label": "Persist local changes"
       },
-      "sync-warning": "To modify this synchronized user, enable local change persistence. Otherwise, your changes will be overwritten during the next synchronization.",
-      "confirm-delete": "Are you sure you want to delete user '{id}'?"
+      "sync-warning": "To modify this synchronized user, enable local change persistence. Otherwise, your changes will be overwritten during the next synchronization."
     },
     "interface-view": {
       "headline": "Config for Interface:"
@@ -504,8 +503,8 @@
           "placeholder": "Persistent Keepalive (0 = default)"
         }
       },
-      "button-apply-defaults": "Apply Peer Defaults",
-      "confirm-delete": "Are you sure you want to delete interface '{id}'?"
+
+      "button-apply-defaults": "Apply Peer Defaults"
     },
     "peer-view": {
       "headline-peer": "Peer:",
@@ -627,8 +626,7 @@
       },
       "expires-at": {
         "label": "Expiry date"
-      },
-      "confirm-delete": "Are you sure you want to delete peer '{id}'?"
+      }
     },
     "peer-multi-create": {
       "headline-peer": "Create multiple peers",

frontend/src/lang/translations/fr.json

@@ -126,7 +126,9 @@
     "peer-expiring": "Le pair expire le",
     "peer-connected": "Connecté",
     "peer-not-connected": "Non connecté",
-    "peer-handshake": "Dernière négociation :"
+    "peer-handshake": "Dernière négociation :",
+    "button-show-peer": "Afficher le pair",
+    "button-edit-peer": "Modifier le pair"
   },
   "users": {
     "headline": "Administration des utilisateurs",
@@ -262,8 +264,7 @@
       },
       "admin": {
         "label": "Est Admin"
-      },
-      "confirm-delete": "Voulez-vous vraiment supprimer l'utilisateur \"{id}\" ?"
+      }
     },
     "interface-view": {
       "headline": "Configuration pour l'interface :"
@@ -376,8 +377,7 @@
           "placeholder": "Persistent Keepalive (0 = par défaut)"
         }
       },
-      "button-apply-defaults": "Appliquer les valeurs par défaut des pairs",
-      "confirm-delete": "Voulez-vous vraiment supprimer l'interface \"{id}\" ?"
+      "button-apply-defaults": "Appliquer les valeurs par défaut des pairs"
     },
     "peer-view": {
       "headline-peer": "Pair :",
@@ -493,8 +493,7 @@
       },
       "expires-at": {
         "label": "Date d'expiration"
-      },
-      "confirm-delete": "Voulez-vous vraiment supprimer le pair \"{id}\" ?"
+      }
     },
     "peer-multi-create": {
       "headline-peer": "Créer plusieurs pairs",

frontend/src/lang/translations/ko.json

@@ -282,7 +282,6 @@
           "label": "관리자 여부"
         }
       },
-      "confirm-delete": "사용자 '{id}'를 삭제하시겠습니까?",
       "interface-view": {
         "headline": "인터페이스 구성:"
       },
@@ -394,8 +393,7 @@
             "placeholder": "영구 Keepalive (0 = 기본값)"
           }
         },
-        "button-apply-defaults": "피어 기본값 적용",
-        "confirm-delete": "인터페이스 '{id}'를 삭제하시겠습니까?"
+        "button-apply-defaults": "피어 기본값 적용"
       },
       "peer-view": {
         "headline-peer": "피어:",
@@ -511,8 +509,7 @@
         },
         "expires-at": {
           "label": "만료 날짜"
-        },
-        "confirm-delete": "피어 '{id}'를 삭제하시겠습니까?"
+        }
       },
       "peer-multi-create": {
         "headline-peer": "여러 피어 생성",

frontend/src/lang/translations/pt.json

@@ -300,8 +300,7 @@
       },
       "admin": {
         "label": "É Administrador"
-      },
-      "confirm-delete": "Tem certeza que deseja excluir o utilizador '{id}'?"
+      }
     },
     "interface-view": {
       "headline": "Configuração para a Interface:"
@@ -414,8 +413,7 @@
           "placeholder": "Keepalive persistente (0 = padrão)"
         }
       },
-      "button-apply-defaults": "Aplicar Padrões de Peer",
-      "confirm-delete": "Tem certeza que deseja excluir a interface '{id}'?"
+      "button-apply-defaults": "Aplicar Padrões de Peer"
     },
     "peer-view": {
       "headline-peer": "Peer:",
@@ -532,8 +530,7 @@
       },
       "expires-at": {
         "label": "Data de expiração"
-      },
-      "confirm-delete": "Tem certeza que deseja excluir o par '{id}'?"
+      }
     },
     "peer-multi-create": {
       "headline-peer": "Criar múltiplos peers",

frontend/src/lang/translations/ru.json

@@ -259,16 +259,16 @@
     "headline-preshared-key": "Новый общий ключ",
     "button-generate": "Генерировать",
     "private-key": {
-      "label": "Приватный ключ",
-      "placeholder": "Приватный ключ"
+        "label": "Приватный ключ",
+        "placeholder": "Приватный ключ"
     },
     "public-key": {
-      "label": "Публичный ключ",
-      "placeholder": "Публичный ключ"
+        "label": "Публичный ключ",
+        "placeholder": "Публичный ключ"
     },
     "preshared-key": {
-      "label": "Общий ключ",
-      "placeholder": "Общий ключ"
+        "label": "Общий ключ",
+        "placeholder": "Общий ключ"
     }
   },
   "calculator": {
@@ -277,18 +277,18 @@
     "headline-allowed-ip": "Новые разрешенные IP-адреса",
     "button-exclude-private": "Исключить частные диапазоны IP-адресов",
     "allowed-ip": {
-      "label": "Разрешенные IP-адреса",
-      "placeholder": "0.0.0.0/0, ::/0",
-      "empty": "Поле ввода не должно быть пустым"
+        "label": "Разрешенные IP-адреса",
+        "placeholder": "0.0.0.0/0, ::/0",
+        "empty": "Поле ввода не должно быть пустым"
     },
     "dissallowed-ip": {
-      "label": "Запрещенные IP-адреса",
-      "placeholder": "10.0.0.0/8, 192.168.0.0/16",
-      "invalid": "Некорректный адрес: {addr}"
+        "label": "Запрещенные IP-адреса",
+        "placeholder": "10.0.0.0/8, 192.168.0.0/16",
+        "invalid": "Некорректный адрес: {addr}"
     },
     "new-allowed-ip": {
-      "label": "Разрешенные IP-адреса",
-      "placeholder": ""
+        "label": "Разрешенные IP-адреса",
+        "placeholder": ""
     }
   },
   "modals": {
@@ -366,8 +366,7 @@
       },
       "admin": {
         "label": "Является администратором"
-      },
-      "confirm-delete": "Вы уверены, что хотите удалить пользователя «{id}»?"
+      }
     },
     "interface-view": {
       "headline": "Конфигурация интерфейса:"
@@ -485,8 +484,7 @@
           "placeholder": "Постоянное поддержание активности (0 = значение по умолчанию)"
         }
       },
-      "button-apply-defaults": "Применить настройки пира по умолчанию",
-      "confirm-delete": "Вы уверены, что хотите удалить интерфейс «{id}»?"
+      "button-apply-defaults": "Применить настройки пира по умолчанию"
     },
     "peer-view": {
       "headline-peer": "Пир:",
@@ -607,8 +605,7 @@
       },
       "expires-at": {
         "label": "Дата истечения срока действия"
-      },
-      "confirm-delete": "Вы уверены, что хотите удалить пир «{id}»?"
+      }
     },
     "peer-multi-create": {
       "headline-peer": "Создать несколько узлов",

frontend/src/lang/translations/uk.json

@@ -151,6 +151,7 @@
     "admin": "Користувач має адміністративні привілеї",
     "no-admin": "Користувач не має адміністративних привілеїв"
   },
+
   "profile": {
     "headline": "Мої VPN-піри",
     "table-heading": {
@@ -188,6 +189,7 @@
       "api-link": "Документація API"
     }
   },
+
   "modals": {
     "user-view": {
       "headline": "Обліковий запис користувача:",
@@ -262,8 +264,7 @@
       },
       "admin": {
         "label": "Адміністратор"
-      },
-      "confirm-delete": "Ви впевнені, що хочете видалити користувача «{id}»?"
+      }
     },
     "interface-view": {
       "headline": "Конфігурація для інтерфейсу:"
@@ -376,8 +377,7 @@
           "placeholder": "Постійний Keepalive (0 = за замовчуванням)"
         }
       },
-      "button-apply-defaults": "Застосувати значення за замовчуванням для пірів",
-      "confirm-delete": "Ви впевнені, що хочете видалити інтерфейс «{id}»?"
+      "button-apply-defaults": "Застосувати значення за замовчуванням для пірів"
     },
     "peer-view": {
       "headline-peer": "Пір:",
@@ -493,8 +493,7 @@
       },
       "expires-at": {
         "label": "Дата закінчення терміну дії"
-      },
-      "confirm-delete": "Ви впевнені, що хочете видалити пір «{id}»?"
+      }
     },
     "peer-multi-create": {
       "headline-peer": "Створити декілька пір",

frontend/src/lang/translations/vi.json

@@ -240,8 +240,7 @@
       },
       "admin": {
         "label": "Là Quản trị viên"
-      },
-      "confirm-delete": "Ban co chac muon xoa nguoi dung '{id}' khong?"
+      }
     },
     "interface-view": {
       "headline": "Cấu hình cho Giao diện:"
@@ -354,8 +353,8 @@
           "placeholder": "Giữ kết nối liên tục (0 = mặc định)"
         }
       },
-      "button-apply-defaults": "Áp dụng Cài đặt Mặc định của Peer",
-      "confirm-delete": "Ban co chac muon xoa giao dien '{id}' khong?"
+
+      "button-apply-defaults": "Áp dụng Cài đặt Mặc định của Peer"
     },
     "peer-view": {
       "headline-peer": "Peer:",
@@ -471,8 +470,7 @@
       },
       "expires-at": {
         "label": "Ngày hết hạn"
-      },
-      "confirm-delete": "Ban co chac muon xoa peer '{id}' khong?"
+      }
     },
     "peer-multi-create": {
       "headline-peer": "Tạo nhiều peer",

frontend/src/lang/translations/zh.json

@@ -242,7 +242,6 @@
           "label": "管理员"
         }
       },
-      "confirm-delete": "确定要删除用户“{id}”吗？",
       "interface-view": {
         "headline": "接口配置: "
       },
@@ -354,8 +353,7 @@
             "placeholder": "持久保持连接 (0 = 默认)"
           }
         },
-        "button-apply-defaults": "应用节点默认值",
-        "confirm-delete": "确定要删除接口“{id}”吗？"
+        "button-apply-defaults": "应用节点默认值"
       },
       "peer-view": {
         "headline-peer": "节点: ",
@@ -471,8 +469,7 @@
         },
         "expires-at": {
           "label": "过期日期"
-        },
-        "confirm-delete": "确定要删除对等点“{id}”吗？"
+        }
       },
       "peer-multi-create": {
         "headline-peer": "创建多个节点",

internal/adapters/database.go

@@ -482,7 +482,7 @@ func (r *SqlRepo) getOrCreateInterface(
 		Identifier: id,
 	}
 
-	err := tx.Preload("Addresses").Attrs(interfaceDefaults).FirstOrCreate(&in, id).Error
+	err := tx.Attrs(interfaceDefaults).FirstOrCreate(&in, id).Error
 	if err != nil {
 		return nil, err
 	}
@@ -691,7 +691,7 @@ func (r *SqlRepo) getOrCreatePeer(ui *domain.ContextUserInfo, tx *gorm.DB, id do
 		Identifier: id,
 	}
 
-	err := tx.Preload("Addresses").Attrs(interfaceDefaults).FirstOrCreate(&peer, id).Error
+	err := tx.Attrs(interfaceDefaults).FirstOrCreate(&peer, id).Error
 	if err != nil {
 		return nil, err
 	}

internal/adapters/database_simple_test.go

@@ -1,73 +0,0 @@
-package adapters
-
-import (
-	"context"
-	"reflect"
-	"testing"
-
-	"github.com/glebarez/sqlite"
-	"github.com/h44z/wg-portal/internal/config"
-	"github.com/h44z/wg-portal/internal/domain"
-	"github.com/stretchr/testify/require"
-	"gorm.io/gorm"
-	"gorm.io/gorm/schema"
-)
-
-func init() {
-	schema.RegisterSerializer("encstr", dummySerializer{})
-}
-
-type dummySerializer struct{}
-
-func (dummySerializer) Scan(ctx context.Context, field *schema.Field, dst reflect.Value, dbValue any) error {
-	return nil
-}
-
-func (dummySerializer) Value(ctx context.Context, field *schema.Field, dst reflect.Value, fieldValue any) (any, error) {
-	if fieldValue == nil {
-		return nil, nil
-	}
-	if v, ok := fieldValue.(string); ok {
-		return v, nil
-	}
-	if v, ok := fieldValue.(domain.PreSharedKey); ok {
-		return string(v), nil
-	}
-	return fieldValue, nil
-}
-
-func TestSqlRepo_SaveInterface_Simple(t *testing.T) {
-	// Initialize in-memory database
-	db, err := gorm.Open(sqlite.Open("file::memory:?cache=shared"), &gorm.Config{})
-	require.NoError(t, err)
-
-	// Migrate only what's needed for this test (avoids Peer and its encstr serializer)
-	require.NoError(t, db.AutoMigrate(&domain.Interface{}, &domain.Cidr{}))
-
-	repo := &SqlRepo{db: db, cfg: &config.Config{}}
-	ctx := domain.SetUserInfo(context.Background(), domain.SystemAdminContextUserInfo())
-	ifaceId := domain.InterfaceIdentifier("wg0")
-
-	// 1. Create an interface with one address
-	addr, _ := domain.CidrFromString("10.0.0.1/24")
-	initialIface := &domain.Interface{
-		Identifier: ifaceId,
-		Addresses:  []domain.Cidr{addr},
-	}
-	require.NoError(t, db.Create(initialIface).Error)
-
-	// 2. Perform a "partial" update using SaveInterface (this is the buggy path)
-	err = repo.SaveInterface(ctx, ifaceId, func(in *domain.Interface) (*domain.Interface, error) {
-		in.DisplayName = "New Name"
-		return in, nil
-	})
-	require.NoError(t, err)
-
-	// 3. Verify that the address was NOT deleted
-	var finalIface domain.Interface
-	require.NoError(t, db.Preload("Addresses").First(&finalIface, "identifier = ?", ifaceId).Error)
-	
-	require.Equal(t, "New Name", finalIface.DisplayName)
-	require.Len(t, finalIface.Addresses, 1, "Address list should still have 1 entry!")
-	require.Equal(t, "10.0.0.1/24", finalIface.Addresses[0].Cidr)
-}