@synopackageland/cli 0.4.0 → 0.5.0

package/dist/templates/native-go/backend/internal/helper/status.go

@@ -25,6 +25,7 @@ type Inventory struct {
 	Protocol       int    `json:"protocol"`
 	PayloadDigest  string `json:"payloadDigest"`
 	RegistryDigest string `json:"registryDigest"`
+	DataAccess     string `json:"dataAccess,omitempty"`
 }
 
 var digestPattern = regexp.MustCompile(`^[a-f0-9]{64}$`)
@@ -32,13 +33,16 @@ var packagePattern = regexp.MustCompile(`^[A-Za-z][A-Za-z0-9]{1,63}$`)
 var versionPattern = regexp.MustCompile(`^\d+(\.\d+){0,3}-\d{4}$`)
 
 func (i Inventory) Valid() bool {
-	return packagePattern.MatchString(i.Package) && versionPattern.MatchString(i.Version) && i.Protocol > 0 && digestPattern.MatchString(i.PayloadDigest) && digestPattern.MatchString(i.RegistryDigest)
+	mode := i.DataAccess == "" || i.Protocol == DataProtocol && (i.DataAccess == "unavailable" || i.DataAccess == "relaxed")
+	return mode && packagePattern.MatchString(i.Package) && versionPattern.MatchString(i.Version) && (i.Protocol == 1 || i.Protocol == DataProtocol) && digestPattern.MatchString(i.PayloadDigest) && digestPattern.MatchString(i.RegistryDigest)
 }
 
 type Record struct {
-	Inventory Inventory `json:"inventory"`
-	Enabled   bool      `json:"enabled"`
-	Residual  bool      `json:"residual,omitempty"`
+	Schema    int        `json:"schema,omitempty"`
+	Lifecycle *Lifecycle `json:"lifecycle,omitempty"`
+	Inventory Inventory  `json:"inventory"`
+	Enabled   bool       `json:"enabled"`
+	Residual  bool       `json:"residual,omitempty"`
 }
 type Request struct {
 	Inventory Inventory `json:"inventory"`
@@ -115,6 +119,9 @@ func LocalClient(i Inventory) (Client, error) {
 
 func (c Client) CheckStopped(ctx context.Context) error {
 	s := c.Status(ctx)
+	if s.State == "incompatible" && s.Reason == "inventory-mismatch" && c.staleManagementStopped() {
+		return nil
+	}
 	if s.Cleanup != "" || (s.State != "not-enabled" && s.State != "stopped" && s.State != "needs-approval") {
 		return errors.New("approved helper teardown required")
 	}
@@ -161,14 +168,16 @@ func readRecord(dir string, owner uint32) (Record, error) {
 	d := json.NewDecoder(io.LimitReader(f, 4097))
 	d.DisallowUnknownFields()
 	var decoded struct {
-		Inventory Inventory `json:"inventory"`
-		Enabled   *bool     `json:"enabled"`
-		Residual  bool      `json:"residual,omitempty"`
+		Schema    int        `json:"schema,omitempty"`
+		Lifecycle *Lifecycle `json:"lifecycle,omitempty"`
+		Inventory Inventory  `json:"inventory"`
+		Enabled   *bool      `json:"enabled"`
+		Residual  bool       `json:"residual,omitempty"`
 	}
-	if d.Decode(&decoded) != nil || d.Decode(new(any)) != io.EOF || !decoded.Inventory.Valid() || decoded.Enabled == nil {
+	if d.Decode(&decoded) != nil || d.Decode(new(any)) != io.EOF || !decoded.Inventory.Valid() || decoded.Enabled == nil || !decoded.Lifecycle.valid() || (decoded.Schema != 0 && decoded.Schema != 2) {
 		return r, errTrust
 	}
-	r = Record{Inventory: decoded.Inventory, Enabled: *decoded.Enabled, Residual: decoded.Residual}
+	r = Record{Schema: decoded.Schema, Inventory: decoded.Inventory, Enabled: *decoded.Enabled, Residual: decoded.Residual, Lifecycle: decoded.Lifecycle}
 	return r, nil
 }
 
@@ -201,6 +210,26 @@ func (c Client) Status(ctx context.Context) State {
 		s.Cleanup = "required"
 		return s
 	}
+	if err == nil && r.Lifecycle != nil && r.Lifecycle.Phase != "running" {
+		if e := protected(c.RunDir, c.Owner, true); e != nil && !errors.Is(e, fs.ErrNotExist) {
+			return result("incompatible", "untrusted-runtime")
+		}
+		if r.Lifecycle.Completion != "" {
+			ok := c.completionPublished(r)
+			if !ok {
+				s := result("failed", "invalid-completion")
+				s.Cleanup = "required"
+				return s
+			}
+		}
+
+		if !runtimeAbsent(c) {
+			s := result("failed", "incomplete-transition")
+			s.Cleanup = "required"
+			return s
+		}
+		return result("stopped", "admin-recover-required")
+	}
 	// Conflict has precedence over missing approval. Even an unapproved leftover
 	// socket is probed before claiming that authorization alone can fix it.
 	if e := protected(c.RunDir, c.Owner, true); e != nil && !errors.Is(e, fs.ErrNotExist) {
@@ -215,7 +244,24 @@ func (c Client) Status(ctx context.Context) State {
 		}
 		state, reason := c.probe(ctx, socket)
 		if state != "ready" {
-			return result(state, reason)
+			s := result(state, reason)
+			if state == "pending" {
+				if err == nil && r.Lifecycle != nil && r.Lifecycle.Phase == "running" {
+					s.PackageRunning = "running"
+					return s
+				}
+				s.State, s.Reason = "failed", "health-refused"
+			}
+			if r.Lifecycle != nil {
+				if _, err := readProcess(c.RunDir, c.Owner); err == nil {
+					s.State = "failed"
+					s.Cleanup = "required"
+				}
+			}
+			if reason == "lifecycle-fenced" {
+				s.Cleanup = "required"
+			}
+			return s
 		}
 	} else if !errors.Is(e, fs.ErrNotExist) {
 		return result("failed", "runtime-unreadable")
@@ -226,7 +272,11 @@ func (c Client) Status(ctx context.Context) State {
 	if e != nil {
 		return result("stopped", "runtime-missing")
 	}
-	return result("ready", "live-handshake")
+	s := result("ready", "live-handshake")
+	if r.Lifecycle != nil {
+		s.PackageRunning = "running"
+	}
+	return s
 }
 
 func (c Client) probe(ctx context.Context, socket string) (string, string) {
@@ -264,8 +314,30 @@ func (c Client) probe(ctx context.Context, socket string) (string, string) {
 	if res.Nonce != req.Nonce || res.Inventory != c.Expected || res.Error == "incompatible" {
 		return "incompatible", "handshake-mismatch"
 	}
+	if !res.OK && (res.Error == "reload-revalidating" || res.Error == "query-revalidating") {
+		return "pending", res.Error
+	}
+	if !res.OK && res.Error == "fenced" {
+		return "failed", "lifecycle-fenced"
+	}
 	if !res.OK || res.Error != "" {
 		return "failed", "health-refused"
 	}
 	return "ready", "live-handshake"
 }
+
+// completionPublished preserves the private proof boundary for diagnostic callers.
+func (c Client) completionPublished(r Record) bool {
+	path := filepath.Join(c.StateDir, "completion-"+r.Lifecycle.Epoch+".json")
+	st, e := os.Lstat(path)
+	ok := e == nil && st.Mode().IsRegular() && st.Mode().Perm() == 0600
+	if ok {
+		s, valid := st.Sys().(*syscall.Stat_t)
+		ok = valid && s.Uid == c.Owner
+	}
+	if ok && uint32(os.Geteuid()) == c.Owner {
+		_, e = completionRecord(c, r)
+		ok = e == nil
+	}
+	return ok
+}

package/dist/templates/native-go/backend/internal/helper/status_test.go

@@ -203,3 +203,131 @@ func TestProcessRecordIsNotReadableByOrdinaryOSUsers(t *testing.T) {
 		t.Fatalf("process record permissions: %v %v", s, err)
 	}
 }
+
+func TestS8InventoryHealthRejectsUnknownProtocol(t *testing.T) {
+	c := fixture(t)
+	for _, protocol := range []int{1, 2} {
+		i := c.Expected
+		i.Protocol = protocol
+		if !i.Valid() || !Respond(i, false, Request{Inventory: i, Op: "health", Nonce: strings.Repeat("c", 64)}).OK {
+			t.Fatal("supported protocol", protocol)
+		}
+	}
+	for _, protocol := range []int{0, 3, 99} {
+		i := c.Expected
+		i.Protocol = protocol
+		res := Respond(i, false, Request{Inventory: i, Op: "health", Nonce: strings.Repeat("c", 64)})
+		if i.Valid() || res.OK || res.Error != "incompatible" {
+			t.Fatal("unknown protocol accepted", protocol, res)
+		}
+	}
+}
+
+func TestPendingHealthRequiresTrustedRunningApproval(t *testing.T) {
+	for _, kind := range []string{"running", "missing", "legacy", "disabled", "residual", "fenced", "nonce", "inventory", "contradictory"} {
+		t.Run(kind, func(t *testing.T) {
+			c := fixture(t)
+			r := Record{Inventory: c.Expected, Enabled: true, Lifecycle: testLifecycle("running")}
+			switch kind {
+			case "legacy":
+				r.Lifecycle = nil
+			case "disabled":
+				r.Enabled = false
+			case "residual":
+				r.Residual = true
+			case "fenced":
+				r.Lifecycle.Phase = "fenced"
+			}
+			if kind != "missing" {
+				record(t, c, r)
+			}
+			live(t, c, c.Expected, func(req Request) Response {
+				res := Response{Inventory: c.Expected, Nonce: req.Nonce, Error: "reload-revalidating"}
+				switch kind {
+				case "nonce":
+					res.Nonce = "replay"
+				case "inventory":
+					res.Inventory.Protocol++
+				case "contradictory":
+					res.OK = true
+				}
+				return res
+			})
+			got := c.Status(context.Background())
+			if kind == "running" {
+				if got.State != "pending" || got.Reason != "reload-revalidating" || got.Cleanup != "" || got.PackageRunning != "running" {
+					t.Fatalf("%+v", got)
+				}
+				if c.CheckStopped(context.Background()) == nil {
+					t.Fatal("pending accepted as stopped")
+				}
+			} else if got.State == "pending" || got.State == "ready" {
+				t.Fatalf("untrusted pending: %+v", got)
+			}
+		})
+	}
+}
+
+func TestS1RInventoryModeIsProtocolTwoOnly(t *testing.T) {
+	i := Inventory{Package: "Fixture", Version: "1.0.0-0001", Protocol: 2, PayloadDigest: strings.Repeat("a", 64), RegistryDigest: strings.Repeat("b", 64)}
+	for _, mode := range []string{"", "unavailable", "relaxed"} {
+		i.DataAccess = mode
+		if !i.Valid() {
+			t.Fatal("known mode refused", mode)
+		}
+	}
+	i.DataAccess = "active"
+	if i.Valid() {
+		t.Fatal("unknown mode accepted")
+	}
+	i.DataAccess = "relaxed"
+	i.Protocol = 1
+	if i.Valid() {
+		t.Fatal("protocol 1 accepted relaxed")
+	}
+}
+
+func TestQueryPendingHealthRequiresTrustedRunningApproval(t *testing.T) {
+	for _, kind := range []string{"running", "missing", "legacy", "disabled", "residual", "fenced", "nonce", "inventory", "contradictory"} {
+		t.Run(kind, func(t *testing.T) {
+			c := fixture(t)
+			r := Record{Inventory: c.Expected, Enabled: true, Lifecycle: testLifecycle("running")}
+			switch kind {
+			case "legacy":
+				r.Lifecycle = nil
+			case "disabled":
+				r.Enabled = false
+			case "residual":
+				r.Residual = true
+			case "fenced":
+				r.Lifecycle.Phase = "fenced"
+			}
+			if kind != "missing" {
+				record(t, c, r)
+			}
+			live(t, c, c.Expected, func(req Request) Response {
+				res := Response{Inventory: c.Expected, Nonce: req.Nonce, Error: "query-revalidating"}
+				switch kind {
+				case "nonce":
+					res.Nonce = "replay"
+				case "inventory":
+					res.Inventory.Protocol++
+				case "contradictory":
+					res.OK = true
+				}
+				return res
+			})
+			got := c.Status(context.Background())
+			if kind == "running" {
+				if got.State != "pending" || got.Reason != "query-revalidating" || got.Cleanup != "" || got.PackageRunning != "running" {
+					t.Fatalf("%+v", got)
+				}
+				if c.CheckStopped(context.Background()) == nil {
+					t.Fatal("pending accepted as stopped")
+				}
+			} else if got.State == "pending" || got.State == "ready" {
+				t.Fatalf("untrusted pending: %+v", got)
+			}
+		})
+	}
+}

package/dist/templates/native-go/backend/internal/helper/supervision.go

@@ -70,15 +70,38 @@ func stopChild(ctx context.Context, child supervisedChild, grace time.Duration)
 // Read to actual connection EOF. A LimitReader's synthetic EOF at the size
 // cap can never certify supervisor exit, so reaching that cap is a refusal.
 func readSupervisorReply(reader io.Reader, want Inventory, nonce string) error {
-	b, err := io.ReadAll(io.LimitReader(reader, 4097))
-	if err != nil || len(b) > 4096 {
+	res, err := decodeSupervisorReply(reader, want, nonce)
+	if err != nil || !res.OK || res.Error != "" {
 		return errTrust
 	}
+	return nil
+}
+
+// Parsing authenticates the entire bounded reply; readiness and stop ACKs
+// retain separate acceptance rules. No refusal can certify supervisor exit.
+func decodeSupervisorReply(reader io.Reader, want Inventory, nonce string) (Response, error) {
 	var res Response
+	b, err := io.ReadAll(io.LimitReader(reader, 4097))
+	if err != nil || len(b) > 4096 {
+		return res, errTrust
+	}
 	d := json.NewDecoder(bytes.NewReader(b))
 	d.DisallowUnknownFields()
-	if d.Decode(&res) != nil || d.Decode(new(any)) != io.EOF || !res.OK || res.Error != "" || res.Inventory != want || res.Nonce != nonce {
-		return errTrust
+	if d.Decode(&res) != nil || d.Decode(new(any)) != io.EOF || res.Inventory != want || res.Nonce != nonce {
+		return Response{}, errTrust
 	}
-	return nil
+	return res, nil
+}
+
+// A pending health refusal cannot be used as a successful admission or stop ACK.
+func supervisorReadiness(res Response, err error) string {
+	if err == nil {
+		if res.OK && res.Error == "" {
+			return "ready"
+		}
+		if !res.OK && (res.Error == "reload-revalidating" || res.Error == "query-revalidating") {
+			return res.Error
+		}
+	}
+	return "fenced"
 }

package/dist/templates/native-go/backend/internal/helper/supervisor_frame.go

@@ -0,0 +1,79 @@
+//go:build !synopkgland_sdk_workers
+
+//line dsmapp/internal/helper/supervisor_frame.go:1
+package helper
+
+import (
+	"bufio"
+	"bytes"
+	"encoding/json"
+	"io"
+	"net"
+	"time"
+)
+
+type supervisorRequest struct {
+	Process Process         `json:"process"`
+	Op      string          `json:"op"`
+	Nonce   string          `json:"nonce"`
+	Data    json.RawMessage `json:"data,omitempty"`
+}
+
+// Legacy M1 requests do not half-close; grant controls MUST half-close. Count
+// the entire v2 frame (including whitespace), reject duplicates and trailing
+// frames, and bound both time and memory. The outer process binds this socket.
+func decodeSupervisorRequest(reader io.Reader) (supervisorRequest, Code) {
+	var req supervisorRequest
+	limited := &io.LimitedReader{R: reader, N: MaxControlBytes + 1}
+	d := json.NewDecoder(limited)
+	var raw json.RawMessage
+	if d.Decode(&raw) != nil {
+		return req, ErrInvalid
+	}
+	if code := decodeDataObject(bytes.NewReader(raw), &req, MaxControlBytes); code != "" {
+		return req, code
+	}
+	if req.Op == "grants" {
+		rest, err := io.ReadAll(io.MultiReader(d.Buffered(), limited))
+		if err != nil {
+			return req, ErrInvalid
+		}
+		if limited.N <= 0 {
+			return req, ErrLimit
+		}
+		if len(bytes.TrimSpace(rest)) != 0 {
+			return req, ErrInvalid
+		}
+	} else if d.InputOffset() > 4096 || len(req.Data) > 0 {
+		return req, ErrInvalid
+	}
+	if !allReferences(req.Nonce) || (req.Op != "admit" && req.Op != "stop" && req.Op != "grants") {
+		return req, ErrInvalid
+	}
+	return req, ""
+}
+
+const supervisorAttestationWait = time.Second
+const supervisorFrameBudget = 100 * time.Millisecond
+
+// The caller sends no frame bytes until its full post-connect hash succeeds.
+// Separate the bounded wait for that computation from the fixed frame budget.
+func receiveSupervisorRequest(conn net.Conn) (supervisorRequest, Code) {
+	firstDeadline := time.Now().Add(supervisorAttestationWait)
+	if err := conn.SetReadDeadline(firstDeadline); err != nil {
+		return supervisorRequest{}, ErrInvalid
+	}
+	reader := bufio.NewReader(conn)
+	if _, err := reader.Peek(1); err != nil || !time.Now().Before(firstDeadline) {
+		return supervisorRequest{}, ErrInvalid
+	}
+	frameDeadline := time.Now().Add(supervisorFrameBudget)
+	if err := conn.SetDeadline(frameDeadline); err != nil {
+		return supervisorRequest{}, ErrInvalid
+	}
+	req, code := decodeSupervisorRequest(reader)
+	if !time.Now().Before(frameDeadline) {
+		return supervisorRequest{}, ErrInvalid
+	}
+	return req, code
+}

package/dist/templates/native-go/backend/internal/helper/supervisor_frame_linux_test.go

@@ -0,0 +1,69 @@
+package helper
+
+import (
+	"context"
+	"encoding/json"
+	"net"
+	"path/filepath"
+	"strings"
+	"testing"
+	"time"
+)
+
+func TestLinuxSupervisorFrameWaitPreservesStopFence(t *testing.T) {
+	p := linuxFixture(t)
+	process := startVMHelper(t, p)
+	conn, err := net.Dial("unix", filepath.Join(p.RunDir, "supervisor.sock"))
+	if err != nil {
+		t.Fatal(err)
+	}
+	defer conn.Close()
+	time.Sleep(50 * time.Millisecond)
+	vmUnit(t, "stop")
+	ctx, cancel := context.WithTimeout(context.Background(), 4*time.Second)
+	defer cancel()
+	if err = waitHealthy(ctx, func(context.Context) bool {
+		r, e := readRecord(p.StateDir, 0)
+		return e == nil && r.Lifecycle.Phase == "stopped" && r.Lifecycle.Completion != ""
+	}); err != nil {
+		t.Fatal("stalled peer prevented owned fence/completion", err)
+	}
+	record, err := readRecord(p.StateDir, 0)
+	if err != nil {
+		t.Fatal(err)
+	}
+	proof, err := completionRecord(p.Client, record)
+	if err != nil || proof.Process != process {
+		t.Fatal("wrong completion", err)
+	}
+	t.Logf("stalled root peer lifecycle reason=%s", record.Lifecycle.Reason)
+	vmUnit(t, "start")
+	if p.Client.Status(context.Background()).State == "ready" {
+		t.Fatal("later active revived stopped epoch")
+	}
+}
+
+func TestLinuxSupervisorRejectsForeignFrameAfterWait(t *testing.T) {
+	p := linuxFixture(t)
+	process := startVMHelper(t, p)
+	conn, err := net.Dial("unix", filepath.Join(p.RunDir, "supervisor.sock"))
+	if err != nil {
+		t.Fatal(err)
+	}
+	defer conn.Close()
+	time.Sleep(150 * time.Millisecond)
+	foreign := process
+	foreign.Start = "foreign"
+	req := supervisorRequest{Process: foreign, Op: "admit", Nonce: strings.Repeat("a", 64)}
+	conn.SetDeadline(time.Now().Add(time.Second))
+	if err = json.NewEncoder(conn).Encode(req); err != nil {
+		t.Fatal(err)
+	}
+	var response Response
+	if err = json.NewDecoder(conn).Decode(&response); err == nil || response.OK {
+		t.Fatal("foreign Process frame accepted", response)
+	}
+	if p.Client.Status(context.Background()).State != "ready" {
+		t.Fatal("foreign frame changed valid epoch")
+	}
+}

package/dist/templates/native-go/backend/internal/helper/supervisor_frame_test.go

@@ -0,0 +1,107 @@
+package helper
+
+import (
+	"encoding/json"
+	"net"
+	"strings"
+	"testing"
+	"time"
+)
+
+func TestSupervisorReceivePhases(t *testing.T) {
+	req := supervisorRequest{Op: "grants", Nonce: strings.Repeat("a", 64), Data: json.RawMessage(`{}`)}
+	frame, _ := json.Marshal(req)
+	for _, kind := range []string{"delayed-attestation", "late-attestation", "buffered-large", "empty-eof", "silent", "partial", "drip", "trailing", "oversize", "duplicate", "valid"} {
+		t.Run(kind, func(t *testing.T) {
+			server, client := supervisorSocketPair(t)
+			defer server.Close()
+			defer client.Close()
+			// Legacy server starts its receive clock before the caller's full hash.
+			if err := server.SetDeadline(time.Now().Add(100 * time.Millisecond)); err != nil {
+				t.Fatal(err)
+			}
+			done := make(chan struct{})
+			go func() {
+				defer close(done)
+				switch kind {
+				case "silent":
+					return
+				case "late-attestation":
+					time.Sleep(1100 * time.Millisecond)
+					client.Write(frame)
+				case "buffered-large":
+					large := req
+					large.Data = json.RawMessage(`{"padding":"` + strings.Repeat("a", 8192) + `"}`)
+					raw, _ := json.Marshal(large)
+					client.Write(raw)
+				case "empty-eof":
+				case "delayed-attestation":
+					time.Sleep(150 * time.Millisecond)
+					client.Write(frame)
+				case "partial":
+					client.Write([]byte("{"))
+					time.Sleep(200 * time.Millisecond)
+				case "drip":
+					for n := 0; n < 8; n++ {
+						if _, e := client.Write([]byte(" ")); e != nil {
+							return
+						}
+						time.Sleep(30 * time.Millisecond)
+					}
+				case "trailing":
+					client.Write(append(frame, []byte(`{}`)...))
+				case "oversize":
+					client.Write(append(frame, []byte(strings.Repeat(" ", MaxControlBytes))...))
+				case "duplicate":
+					client.Write([]byte(`{"op":"grants","op":"grants","nonce":"` + req.Nonce + `","data":{}}`))
+				default:
+					client.Write(frame)
+				}
+				client.CloseWrite()
+			}()
+			started := time.Now()
+			got, code := receiveSupervisorRequest(server)
+			elapsed := time.Since(started)
+			server.Close()
+			client.Close()
+			<-done
+			if kind == "valid" || kind == "delayed-attestation" || kind == "buffered-large" {
+				if code != "" || got.Nonce != req.Nonce {
+					t.Fatalf("valid delayed/buffered frame refused: %s", code)
+				}
+			} else if code == "" {
+				t.Fatal("invalid/stalled frame accepted")
+			}
+			if kind == "silent" && (elapsed < 900*time.Millisecond || elapsed > 1500*time.Millisecond) {
+				t.Fatalf("first-byte bound: %s", elapsed)
+			}
+			if (kind == "partial" || kind == "drip") && elapsed > 180*time.Millisecond {
+				t.Fatalf("frame deadline was extended: %s", elapsed)
+			}
+		})
+	}
+}
+
+func TestSupervisorFrameBudgetAlsoBoundsLegacyReply(t *testing.T) {
+	// Keep the synchronous pipe here: an unread small Unix socket reply fits in
+	// the kernel buffer and would not exercise the bounded blocked-write path.
+	server, client := net.Pipe()
+	defer server.Close()
+	defer client.Close()
+	// A watchdog protects the test if reply writes accidentally lose their bound.
+	server.SetWriteDeadline(time.Now().Add(2 * time.Second))
+	req := supervisorRequest{Op: "admit", Nonce: strings.Repeat("b", 64)}
+	frame, _ := json.Marshal(req)
+	sent := make(chan struct{})
+	go func() { defer close(sent); time.Sleep(150 * time.Millisecond); client.Write(frame) }()
+	if _, code := receiveSupervisorRequest(server); code != "" {
+		t.Fatal(code)
+	}
+	<-sent
+	started := time.Now()
+	_, err := server.Write([]byte(`{"ok":true}`))
+	elapsed := time.Since(started)
+	if e, ok := err.(net.Error); !ok || !e.Timeout() || elapsed < 70*time.Millisecond || elapsed > 350*time.Millisecond {
+		t.Fatalf("unbounded/expired legacy reply: %s %v", elapsed, err)
+	}
+}