feat: Safeguard gateway addition in EditServerModal by implementing error handling during state updates, ensuring robust server configuration management.

2025-12-07 00:58:26 +07:00
parent e14839ca21
commit cb657fe56e
1 changed files with 16 additions and 10 deletions
@@ -630,6 +630,8 @@ function ServerManager() {
    setTimeout(() => setSuccess(''), 3000);
  };

+  const gateways = Array.isArray(localServer.gateways) ? localServer.gateways : [];
+
  return (
    <>
      {error && (
@@ -1538,11 +1540,11 @@ function EditServerModal({ show, server, onChange, onSave, onClose }) {
                <label className="form-label">Тип туннеля</label>
                <input type="text" className="form-control" value={localServer.tunnel || ''} onChange={e => handleChange('tunnel', e.target.value)} />
              </div>
-            {NEEDS_GATEWAYS(localServer.type) && (
+              {NEEDS_GATEWAYS(localServer.type) && (
                <>
                  <div className="form-label">Шлюзы</div>
                  <div className="list-group list-group-flush border rounded mb-2">
-                    {(localServer.gateways || []).map((gw, idx) => (
+                    {gateways.map((gw, idx) => (
                      <div key={gw.id || idx} className="list-group-item">
                        <div className="row g-2 align-items-end">
                          <div className="col-12 col-md-4">
@@ -1634,14 +1636,18 @@ function EditServerModal({ show, server, onChange, onSave, onClose }) {
                    type="button"
                    className="btn btn-outline-primary btn-sm"
                  onClick={() => {
-                    setLocalServer((prev) => {
-                      const list = Array.isArray(prev?.gateways) ? prev.gateways : [];
-                      const nextList = [...list, makeGateway({ primary: list.length === 0 })];
-                      const next = { ...(prev || {}), gateways: nextList };
-                      onChange && onChange(next);
-                      return next;
-                    });
-                  }}
+                      try {
+                        setLocalServer((prev) => {
+                          const list = Array.isArray(prev?.gateways) ? prev.gateways : [];
+                          const nextList = [...list, makeGateway({ primary: list.length === 0 })];
+                          const next = { ...(prev || {}), gateways: nextList };
+                          onChange && onChange(next);
+                          return next;
+                        });
+                      } catch (e) {
+                        console.error('Ошибка при добавлении шлюза в модалке редактирования', e);
+                      }
+                    }}
                  >
                    <IconPlus className="icon" />
                    Добавить шлюз