From cdf0b70e257d55b9903f1e1a63136e57b8b8502b Mon Sep 17 00:00:00 2001 From: Dillard Blom Date: Thu, 29 Oct 2026 02:40:03 +0100 Subject: [PATCH] Changelog: the admin page's single-line key field as the cause of #24; PLAN 1.0.1: #24, database locked at the start of a full restore on SQLite --- CHANGELOG.md | 11 ++++++----- PLAN.md | 10 ++++++++++ 2 files changed, 16 insertions(+), 5 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 782b583..bd7fcd5 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -22,11 +22,12 @@ instead carried by the app description, the git tag, the release, and this file. recorded user export for this location". ### Fixed -- An SFTP location with an RSA key in OpenSSH format ("BEGIN OPENSSH PRIVATE KEY") whose line - breaks were lost, e.g. turned into spaces on the way in, stopped working after an upgrade to - Nextcloud 35 with "Unable to read key": Nextcloud 35 reads keys with a stricter library. The - line breaks are now restored before the key is used, so the stored location works again without - re-entering it (#24). +- An SFTP location with an RSA key in OpenSSH format ("BEGIN OPENSSH PRIVATE KEY") stopped + working after an upgrade to Nextcloud 35 with "Unable to read key", and such a key could not be + added there at all (#24). The "Private key" field on the admin page is a single line, so a + pasted key lost its line breaks; Nextcloud 34 still read it, Nextcloud 35 reads keys with a + stricter library. The line breaks are now restored before the key is used, so a stored location + works again without re-entering it. ## [1.0.0] - 2026-10-29 diff --git a/PLAN.md b/PLAN.md index 93a749a..b90132f 100644 --- a/PLAN.md +++ b/PLAN.md @@ -454,6 +454,16 @@ in README "Known limitations": regel wat erin staat. Nu zijn ze alleen via de README te vinden. Zichtbaar ook zonder sleutel, zodat de gids vindbaar is vóór de eerste installatie-stap. "NG Backup is in beta" gaat uit de feedbackregel — **klaar** (2026-10-27). + 12. **SFTP-sleutel zonder regeleinden** (issue #24, 2026-10-28): het veld "Private key" op de + beheerpagina is één regel, dus een geplakte sleutel verliest zijn regeleinden. NC34 + (phpseclib 2) las dat nog, NC35 (phpseclib 3) weigert een OpenSSH-sleutel dan met "Unable + to read key". De regeleinden worden nu teruggezet vlak voor gebruik — **klaar** (2026-10-28). + 13. **"database is locked" bij het begin van `backup:restore:full` op SQLite** (CI, 2026-10-26 + bij #22 en 2026-10-29 op `main` na de merge van 1.0.0): de restore stopt meteen na het + aanzetten van de onderhoudsmodus. Niet altijd, dus een tweede proces dat op hetzelfde moment + de database gebruikt (cron, een webverzoek of de opstart van de container). Kan ook bij een + echte restore naar SQLite gebeuren. Uitzoeken wie de lock vasthoudt; dan wachten en opnieuw + proberen (busy timeout) of eerst de andere schrijvers stoppen. - **DR-tests over databasegrenzen** (besluit 2026-10-20; productie nc01 draait PostgreSQL): na de MariaDB→PostgreSQL-fout (tellers, rc5) elke overstap in de integratietest (`to-pgsql`, `to-mysql`, `to-sqlite`, ook in CI). Volle DR-tests met echte data: vóór 1.0.0 de S3-simulatie