32ab2c7f47
Go-Quality-Baseline-Rollout ABGESCHLOSSEN. Code-Quality-Backlog (55 → 0): - errcheck: unbehandelte Close/Rollback/Remove explizit `_ =`; fmt.Sscanf `_, _ =` (Zero-Value degradiert sauber). - unused: toter Code entfernt (nodeIDOrHostname, stripTrailingNewline, acme.Service.user, strFold + ungenutzter Import). - noctx (net/http): http.NewRequestWithContext mit vorhandenem ctx. - staticcheck: QF1001/S1009/ST1005/SA9003. - contextcheck: detached-by-design-Stellen mit begründetem //nolint. Zwei echte Bugs beim Aufräumen gefunden+gefixt: - backup/remote SFTP-Upload: dst.Close()-Flush-Fehler wurde verschluckt → unvollständiges Remote-File galt als Erfolg. Jetzt geprüft+gemeldet. - haproxy_test: leere if-Assertion (SA9003) testete faktisch nichts → echte t.Errorf-Prüfung (kein HSTS für HSTS-disabled Domain). Bewusste Config-Entscheidungen (.golangci.yml): - noctx-on-os/exec ausgeschlossen: System-Command-Reloads (systemctl/nft/ wg/pg) dürfen NICHT an den Request-Context gebunden werden — ein Client- Disconnect darf keinen laufenden Reload mitten in der Ausführung killen. net/http-noctx bleibt voll aktiv. KEINE exec-Zeile im Code angefasst. - rowserrcheck/sqlclosecheck raus (database/sql-Linter, bei pgx nur FPs). Gate scharf gestellt: Makefile release-check ruft golangci-lint jetzt als HARTEN Gate (install-if-missing, pinned v2.12.2). `make release-check` grün: vet, golangci-lint, govulncheck, build, test -race. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
90 lines
2.4 KiB
Go
90 lines
2.4 KiB
Go
package kea
|
|
|
|
import (
|
|
"context"
|
|
"encoding/json"
|
|
"os"
|
|
"os/exec"
|
|
"strings"
|
|
"testing"
|
|
"time"
|
|
|
|
"git.netcell-it.de/projekte/edgeguard-native/internal/database"
|
|
)
|
|
|
|
// Guarded integration test: set EG_FWTEST_DSN (sonst skip).
|
|
func TestRender_DHCPConfig(t *testing.T) {
|
|
dsn := os.Getenv("EG_FWTEST_DSN")
|
|
if dsn == "" {
|
|
t.Skip("set EG_FWTEST_DSN to run the kea renderer test")
|
|
}
|
|
ctx := context.Background()
|
|
var mErr error
|
|
for i := 0; i < 3; i++ {
|
|
if mErr = database.Migrate(ctx, dsn); mErr == nil {
|
|
break
|
|
}
|
|
time.Sleep(700 * time.Millisecond)
|
|
}
|
|
if mErr != nil {
|
|
t.Fatalf("migrate: %v", mErr)
|
|
}
|
|
pool, err := database.Open(ctx, dsn)
|
|
if err != nil {
|
|
t.Fatalf("open: %v", err)
|
|
}
|
|
defer pool.Close()
|
|
|
|
for _, q := range []string{
|
|
`DELETE FROM dhcp_reservations`,
|
|
`DELETE FROM dhcp_subnets`,
|
|
`UPDATE dhcp_settings SET enabled=true, default_lease=3600, max_lease=7200, domain_name='lan', dns_servers='1.1.1.1, 8.8.8.8' WHERE id=1`,
|
|
} {
|
|
if _, err := pool.Exec(ctx, q); err != nil {
|
|
t.Fatalf("seed (%s): %v", q, err)
|
|
}
|
|
}
|
|
var subID int64
|
|
if err := pool.QueryRow(ctx, `
|
|
INSERT INTO dhcp_subnets (name, interface_name, subnet_cidr, pool_start, pool_end, gateway, dns_servers, active)
|
|
VALUES ('lan','eth1','10.0.0.0/24','10.0.0.100','10.0.0.200','10.0.0.1','',true) RETURNING id`).Scan(&subID); err != nil {
|
|
t.Fatalf("seed subnet: %v", err)
|
|
}
|
|
if _, err := pool.Exec(ctx, `
|
|
INSERT INTO dhcp_reservations (subnet_id, mac_address, ip_address, hostname, active)
|
|
VALUES ($1,'aa:bb:cc:dd:ee:ff','10.0.0.50','printer',true)`, subID); err != nil {
|
|
t.Fatalf("seed reservation: %v", err)
|
|
}
|
|
|
|
out, err := New(pool).RenderToString(ctx)
|
|
if err != nil {
|
|
t.Fatalf("render: %v", err)
|
|
}
|
|
if !json.Valid([]byte(out)) {
|
|
t.Fatalf("rendered config is not valid JSON:\n%s", out)
|
|
}
|
|
for _, want := range []string{
|
|
`"10.0.0.0/24"`,
|
|
`"10.0.0.100 - 10.0.0.200"`,
|
|
`"aa:bb:cc:dd:ee:ff"`,
|
|
`"routers"`,
|
|
`"eth1"`,
|
|
`"hw-address"`,
|
|
`"valid-lifetime": 3600`,
|
|
} {
|
|
if !strings.Contains(out, want) {
|
|
t.Errorf("rendered config missing %q\n----\n%s", want, out)
|
|
}
|
|
}
|
|
|
|
// Best-effort: echte Kea-Validierung, falls die Binary da ist.
|
|
if _, statErr := os.Stat(keaBinary); statErr == nil {
|
|
f, _ := os.CreateTemp(t.TempDir(), "kea-*.conf")
|
|
_, _ = f.WriteString(out)
|
|
_ = f.Close()
|
|
if combined, err := exec.Command(keaBinary, "-t", f.Name()).CombinedOutput(); err != nil {
|
|
t.Fatalf("kea-dhcp4 -t rejected rendered config: %v\n%s", err, combined)
|
|
}
|
|
}
|
|
}
|