Changeset 0ad37a0 in Klonkt for src

Timestamp:

09/23/2026 02:59:29 PM (5 hours ago)

Author:

Bart <bart@…>

Branches:

main

Children:

407353e

Parents:

8a474ab

git-author:

Bart <bart@…> (09/23/2026 02:59:10 PM)

git-committer:

Bart <bart@…> (09/23/2026 02:59:29 PM)

Message:

Lege-bericht-controle deed niets, want een lege paragraaf is waar

De voorwaarde luidde !plain.trim() && !object.content && geen bijlagen. Shaer
stuurde bij lege tekst <p></p>, en die string is waar, dus de hele controle
kon voor die client nooit kloppen: een bericht zonder inhoud kwam er gewoon
langs en werd een lege alinea in de tijdlijn.

plain is al uit content afgeleid, dus die tweede term voegde niets toe behalve
dat gat. Weg. Een bijlage mag nog steeds het hele bericht zijn -- dat is precies
wat een foto-post is.

De app stuurt sinds vandaag geen lege body meer mee, maar de controle hoort ook
te werken tegen een client die dat wel doet.

Co-Authored-By: Claude Opus 5 <claude@…>

File:

• src/services/ap-c2s.js (modified) (1 diff)

src/services/ap-c2s.js

@@ -160 +160 @@
         // A picture (or a recording) can be the whole message: media-only
         // notes pass here; c2sCreatePost validates the attachments themselves.
-        if (!plain.trim() && !object.content && !(Array.isArray(object.attachment) && object.attachment.length)) {
+        // `!object.content` stond hier en zette de controle in de praktijk uit:
+        // Shaer stuurde bij lege tekst `<p></p>`, en die string is waar, dus
+        // een leeg bericht kwam er gewoon langs en werd een lege alinea in de
+        // tijdlijn. `plain` is al uit content afgeleid, dus die tweede term
+        // voegde niets toe behalve dat gat. Een bijlage mag nog steeds het hele
+        // bericht zijn -- dat is wat een foto-post is.
+        if (!plain.trim() && !(Array.isArray(object.attachment) && object.attachment.length)) {
           return { status: 400, error: 'empty_note' };
         }