Naprawa: ekran kont tłumaczy awarię magazynu zamiast dawać 500 #71

Merged

gitea merged 1 commits from fix/konta-blad-magazynu into master 2026-08-09 20:42:26 +00:00

Conversation 0 Commits 1 Files Changed 3

+119 -8

gitea commented 2026-08-07 17:37:23 +00:00

Owner

Copy Link

Copy Source

Ekran „Konta" wywalał się na produkcji błędem 500 bez słowa wyjaśnienia.

Przyczyna

_read() łapał wyłącznie brak pliku i niepoprawny JSON. Każdy inny błąd systemu plików — a na udziale NFS to głównie prawa — leciał na wierzch jako nieobsłużony wyjątek.

Odtworzone lokalnie, oba przypadki:

katalog stanu	przed	po
bez prawa odczytu	GET /accounts → 500	200 + Nie mogę odczytać pliku kont /app/state/accounts.json: Permission denied…
tylko do odczytu	POST /accounts/create → 500	303 + Nie mogę zapisać pliku kont…

Dlaczego to szczególnie zły sposób na awarię akurat tutaj

Ekran kont jest jedynym miejscem, z którego administrator może taki problem naprawić. Gołe 500 nie mówi ani co, ani gdzie. Teraz komunikat niesie ścieżkę, powód z systemu operacyjnego i podpowiedź najczęstszej przyczyny.

Szczegóły, które wyszły dopiero z testów

mkstemp musi być w klamrze. Przy katalogu tylko do odczytu wywala się ono pierwsze, jeszcze zanim dojdzie do zapisu i podmiany — obudowanie samego os.replace nic by nie dało. Złapane testem, nie przeglądem kodu.

Uszkodzony plik nie jest nadpisywany. Wcześniej niepoprawny JSON dawał pusty zbiór kont, co przy pierwszym zapisie skasowałoby wszystkie konta bez śladu. Teraz to odmowa z komunikatem, plik zostaje nietknięty — osobny test tego pilnuje.

Weryfikacja

3 nowe testy awarii magazynu. Prezentacja: 290 zielonych.

🤖 Generated with Claude Code

Ekran „Konta" wywalał się na produkcji błędem 500 bez słowa wyjaśnienia. ## Przyczyna `_read()` łapał wyłącznie brak pliku i niepoprawny JSON. Każdy inny błąd systemu plików — a na udziale NFS to głównie **prawa** — leciał na wierzch jako nieobsłużony wyjątek. Odtworzone lokalnie, oba przypadki: | katalog stanu | przed | po | |---|---|---| | bez prawa odczytu | `GET /accounts` → **500** | 200 + `Nie mogę odczytać pliku kont /app/state/accounts.json: Permission denied…` | | tylko do odczytu | `POST /accounts/create` → **500** | 303 + `Nie mogę zapisać pliku kont…` | ## Dlaczego to szczególnie zły sposób na awarię akurat tutaj Ekran kont jest **jedynym miejscem, z którego administrator może taki problem naprawić**. Gołe 500 nie mówi ani co, ani gdzie. Teraz komunikat niesie **ścieżkę**, powód z systemu operacyjnego i podpowiedź najczęstszej przyczyny. ## Szczegóły, które wyszły dopiero z testów **`mkstemp` musi być w klamrze.** Przy katalogu tylko do odczytu wywala się **ono pierwsze**, jeszcze zanim dojdzie do zapisu i podmiany — obudowanie samego `os.replace` nic by nie dało. Złapane testem, nie przeglądem kodu. **Uszkodzony plik nie jest nadpisywany.** Wcześniej niepoprawny JSON dawał pusty zbiór kont, co przy pierwszym zapisie skasowałoby **wszystkie konta bez śladu**. Teraz to odmowa z komunikatem, plik zostaje nietknięty — osobny test tego pilnuje. ## Weryfikacja 3 nowe testy awarii magazynu. Prezentacja: 290 zielonych. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

gitea added 1 commit 2026-08-07 17:37:23 +00:00

fix(konta): awaria magazynu tłumaczy się zamiast dawać gołe 500 ... 5a7f7e1b13

Ekran „Konta" wywalał się na produkcji błędem 500 bez słowa wyjaśnienia.
Odtworzone lokalnie: `_read()` łapał wyłącznie brak pliku i zły JSON, więc każdy
inny błąd systemu plików — a na udziale NFS to głównie prawa — leciał na wierzch
jako nieobsłużony wyjątek.

To jest szczególnie zły sposób na awarię AKURAT TUTAJ: ekran kont jest jedynym
miejscem, z którego administrator może taki problem naprawić, a gołe 500 nie mówi
mu ani co, ani gdzie.

Teraz każdy błąd magazynu ma twarz: osobny wyjątek AccountsUnavailable niosący
ŚCIEŻKĘ i powód z systemu operacyjnego, plus podpowiedź najczęstszej przyczyny
(prawa katalogu na udziale albo wolumen zamontowany tylko do odczytu). Strona
renderuje się normalnie z tym komunikatem u góry.

Objęte są wszystkie cztery drogi zapisu, a nie tylko odczyt. W szczególności
mkstemp: przy katalogu tylko do odczytu wywala się ONO pierwsze, jeszcze zanim
dojdzie do zapisu i podmiany — więc obudowanie samego os.replace nic by nie dało
(złapane testem, nie przeglądem kodu).

USZKODZONY PLIK NIE JEST NADPISYWANY. Wcześniej niepoprawny JSON dawał pusty
zbiór kont, co przy pierwszym zapisie skasowałoby WSZYSTKIE konta bez śladu.
Teraz to odmowa z komunikatem — plik zostaje nietknięty, a test tego pilnuje.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

gitea merged commit e2b50b284f into master 2026-08-09 20:42:26 +00:00

gitea referenced this pull request 2026-08-11 12:40:43 +00:00

Naprawa CI: testy awarii magazynu nie mogą zależeć od praw pliku #73

gitea referenced this issue from a commit 2026-08-11 12:40:44 +00:00

fix(testy): awaria magazynu kont wymuszana podmianą, nie prawami pliku

gitea referenced this issue from a commit 2026-08-11 14:38:15 +00:00

fix(testy): awaria magazynu kont wymuszana podmianą, nie prawami pliku

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

gitea

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: gitea/astrololo#71