You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
qa: harden blacklist test process and runner boundaries #83
Blacklist-QA'ens parameterbinding og schemafejl blev rettet og verificeret i PR #82. Under den fulde SSH/Docker-verifikation blev der fundet nogle eksisterende driftsmæssige svagheder uden for selve SQL-bindingens scope.
Opgave
Undgå at sende test-pubkeyen i procesargumenter i config-edit-SSH, curl-URL'er og lokale grep-kald. Pubkeyen er offentlig, men stdin/filer er et mere robust mønster og gør procesliste-kontrakten entydig.
Afklar eller fjern ADMIN_API_TOKEN-stien. Den kalder i dag /api/admin/transmissions?from_node=..., som ikke findes, og falder derfor altid videre til SQLite. Hvis API-stien bevares, skal den have en virkelig serverkontrakt og tests.
Adskil host- og containersti til SQLite-databasen. Den nuværende fælles TARGET_DB_PATH kan få hostens sqlite3 til at oprette en tom fil, hvis stien kun er gyldig i containeren.
Dæk SIGPIPE, når kun stderr-forbindelsen brydes, eller dokumentér eksplicit hvorfor det ikke understøttes.
Ret den upræcise formulering omkring RESTART_WAIT_S.
Tilføj shellcheck i den relevante lokale/CI-kontrol, hvis det kan gøres uden at ændre workflowets sikkerheds- eller deploysemantik.
Acceptkriterier
Ingen pubkey, token eller SQL-payload findes i argv under en fuld isoleret kørsel.
Admin-API-stien er enten fungerende og kontrakttestet eller fjernet med tydelig SQLite-only dokumentation.
Host/container kan have forskellige DB-stier uden at oprette en utilsigtet tom database.
Alle succes-, fejl-, signal- og teardown-forløb bevarer korrekt exit-status og gendanner config.
Unit-, mutations- og fuld isoleret SSH/Docker-verifikation er grøn.
Hold denne opgave adskilt fra den allerede mergede schema- og parameterbindingsrettelse.
Activity
fix(qa-test): start the SIGPIPE trap cases from the default dispositi…
reviewfix(qa): keep existing blacklist rules, real topology route, ca…
test(qa): mixed-case and python3 pre-blacklisted entries (#83)
reviewfix(qa): one-document topology parse, server-equal blacklist ch…
chore(qa): justify two shellcheck info findings (#83)
dborup commentedon Oct 14, 2026
Parked per owner decision (2026-10-14): this is an internal QA script used rarely. It is kept in the backlog and not scheduled.