One codec for the grants part's memberships state, so its writer and readers cannot drift (issue 774) #134

Merged

jschoubben merged 2 commits from fix/memberships-shape into main 2026-10-30 14:42:05 +00:00

Conversation 1 Commits 2 Files Changed 3

+109 -2

2 Commits

Author	SHA1	Message	Date
jschoubben	83cf97b690	Refuse an old wrapper of a held-back membership, so a reader falls back to the assignments stream ... The grants part's old build wrote its wrapper for every membership, a held-back one too, with the unpublished composition inside; the new part never rewrites such a key, so the decoder took the held-back membership as given. It now reads a wrapper only when its published says published or unchanged. The codec's rows join the round-trip test (mesh-sdk is over its unit budget). Review round 1 of issue 774's fix.	2026-10-30 15:36:23 +01:00
jschoubben	46d4b4da5e	One codec for the grants part's memberships, so its writer and the tool runner cannot drift ... The grants part wrote its own wrapper into mesh-grants_memberships while the tool runner read the value as a bare membership: novox's tool runner served no tool. EncodeGrantsMembership writes the bare membership; DecodeGrantsMembership and BareGrantsMembership read it, still read the old wrapper for the transition, and refuse a value naming no node or module instead of answering an empty membership. Issue 774 (the bus login list, L3). Quick fix: write the bare membership in the grants part alone. Thorough, chosen: one codec in mesh-sdk both sides use, because a hand-built test fixture agreed with the mistake and hid it.	2026-10-30 15:23:17 +01:00