Alphea App

0.8-2 review findings 먹음¶

commit:server2|b69a259

첫 컷 (6ee2aa3) 후 리뷰에서 나온 세 건 (High 1 + Medium 2) 모두 같은 슬라이스에 접어 넣음.

1) High — interceptor chain order. Connect 가 chain 을 outer→inner 로 실행하는데 AuthInterceptor 가 마지막 (innermost) 이었음. CommandAuditInterceptor 가 outer 이라 next() 다른 ctx 를 못 봅. 결과 protected RPC 의 audit row 가 principal 빈 채 들어감. Fix: AuthInterceptor 를 access/audit 보다 outer 로 이동. Metrics + Logging 은 그대로 outermost — auth 실패도 관찰 가능. Public-RPC best-effort 동반: publicRPCs 분기에도 token 있으면 verify-and-attach. 충관 시에는 그냥 anonymous 로 통과 (public RPC 는 token 필요 안 함)

2) Medium — namespace extraction. 첫 컷에서는 perma-NULL + TODO. 이제 모든 dFS / Delta request 가 관습적으로 노출하는 GetNamespace() *DfsNamespaceRef 와 ResolvePathRequest 의 GetUri() 를 interface dispatch 로 읽음. 환경 추출 → "wallet:0xabc" / "token:0xdef" 등. 새 namespace-bearing RPC 도 자동 attribution. Wallet/Node 챌린지는 의도적 NULL — principal_id 가 그 정보 carry

3) Low — DB-free integration tests. 첫 컷 테스트가 helper 는 잘 잡는데 실제 chain output 은 못 잡은 점이 High 버그를 뚫은 원인. 신규:

• TestExtractAuditNamespace (8 sub-case) + TestParseURINamespace (7 sub-case) — namespace 추출 하나하나
• TestInterceptorChainAttributesProtectedRPC — 전체 chain 을 runChain 으로 이어서. channel-backed fakeAuditWriter 로 off-request goroutine 을 결정론적 수신. principal + namespace + surface + command + request_id + args/result hash 전부 검증. High regression 가 다시 들어오면 이 테스트가 잡음
• TestInterceptorChainAttributesPublicRPCWithBearer — public RPC + bearer → audit row 에 principal
• TestInterceptorChainProtectedRPCRejectsAnonymous — anonymous protected call → Unauthenticated short-circuit + 200ms 안 row 안 들어감 검증
• specRequest wrapper — connect.NewRequest 가 비워둘 Spec().Procedure 를 테스트에서 제어

검증¶

go test  ./sdk ./internal/dshcore ./internal/server \
         ./internal/store ./internal/handler \
         ./cmd/dshai ./cmd/dsh ./cmd/sdktest      — all clean
go vet   ./...                                    — clean
go build ./...                                    — clean

decision-log.md 의 2026-05-20 follow-up entry 에 세 건의 "왜 그 첨 구현에서 이 올가미가 나왔는지" 설명. 다음 슬라이스에서 같은 함정 다시 빠지지 않게.

0.8-2 second-round review findings 먹음¶

commit:server2|bbe43db

지난 라운드 (b69a259) 의 chain-order fix 가 protected RPC 의 principal 는 채웠지만, AuthInterceptor 가 outer 가 되어 거절한 RPC (만료 wallet token, bad bearer, missing Authorization) 가 audit 에 아예 안 남는 새 regression 을 만듦. 세 건 (Medium 1 + Low 2) 모두 같은 슬라이스에 접어 넣음.

1) Medium — authentication failures 도 audit 에 남아야. Phase 0.8 audit 의 의미는 "모든 시도된 RPC 가 흔적을 남김" — expired token / bad bearer 도 row 있어야. Fix: chain 순서를 원래대로 (AccessLog / Audit 가 Auth 를 감셜) 되돌리고, audit logger 가 자체 best-effort verify(token) 으로 principal 채움. principalFromAuthorization helper 신규 — 명시적 attribution-only path (permission 결정에 임팝트 없음)

New chain (원래 의도):

MetricsInterceptor          (outer)
LoggingInterceptor
AccessLogInterceptor        — best-effort verify
CommandAuditInterceptor     — best-effort verify
AuthInterceptor             (inner — 유일 게이트)

Anonymous / bad-bearer protected 가 이제 row 남김: status="error", error_code="permission_denied", empty principal (best-effort verify 도 실패), populated namespace + procedure + args_hash. handler 는 여전히 안 실행.

2) Low — namespace scheme + address gating. parseURINamespace 이 알려진 스키마·호스트 필터링 없이 https://example/x 도 namespace="https:example" 로 기록. 컬럼 의미 오염. Fix: isDFSNamespaceScheme(s) 가 wallet/token 만 허용, isHexAddress(s) 가 lowercase 0x + 40 hex 강제. DfsNamespaceRef-holder 도 같은 validation. column 이 이제 엄격하게 "dFS namespace key 또는 NULL"

3) Low — test housekeeping. var _ sync.Mutex placeholder + sync import 제거. 필요해지면 그때 import.

테스트 변경¶

• TestInterceptorChainProtectedRPCRejectsAnonymous (이전) → 제거 (잘못된 assertion). 대체 2개:
  • TestInterceptorChainAuditsAnonymousProtectedRPC — row 가 남고 status=error / error_code=permission_denied / namespace+procedure+args_hash 채움
  • TestInterceptorChainAuditsBadBearerProtectedRPC — expired wallet token 시나리오
• TestInterceptorChainAttributesProtectedRPC 40-hex 주소 fixture 로
• TestExtractAuditNamespace + TestParseURINamespace 에 non-dFS scheme + bad address 케이스 추가

검증¶

go test  ./sdk ./internal/dshcore ./internal/server \
         ./internal/store ./internal/handler \
         ./cmd/dshai ./cmd/dsh ./cmd/sdktest      — all clean
go vet   ./...                                    — clean
go build ./...                                    — clean

decision-log 에 세 번째 2026-05-20 entry — "왜 이전 라운드의 fix 가 틀렸는지". 다음 슬라이스에서 같은 trap 재현하지 않게.