fix: rtcp-feedback-negotiation/#90

[hjbin-25]

변경 내용

송출 시작 직후 몇 초간 클라이언트 미리보기 영상의 정지 영역(벽·천장)만 파괴되고 자가복구되는 증상을 고쳤어요. 디코더 레퍼런스 손실이 원인이에요.

internal/session/video_codec.go의 offeredVideoCodecs가 클라이언트 offer를 파싱해 RTPCodecCapability를 재구성할 때 RTCPFeedback을 채우지 않았어요. 이 값이 prepareOutputForOffer의 SetCodecPreferences로 비디오 트랜시버에 통째로 덮어써지고, Pion은 answer의 a=rtcp-fb 라인을 협상된 코덱의 RTCPFeedback에서 생성하므로 answer에 피드백 라인이 한 줄도 실리지 않았어요.

비디오 m-line은 sendrecv 하나뿐이라 인그레스·이그레스 양방향 모두 NACK · PLI · FIR · transport-cc · goog-remb가 미협상 상태였어요.

이제 offer SDP의 a=rtcp-fb 라인을 페이로드 타입별로 읽어 RTPCodecCapability.RTCPFeedback에 채워요. 서버가 임의로 피드백을 추가하지 않고 클라이언트가 제안한 것만 그대로 되돌려줘요. 코덱 선택 순서(클라이언트 offer 순서)는 바뀌지 않아요.

실측 — answer의 a=rtcp-fb 라인 수:

피드백	VP8 선호 (전 → 후)	H.264 선호 (전 → 후)
nack	0 → 2	0 → 1
nack pli	0 → 2	0 → 1
ccm fir	0 → 1	0 → 1
goog-remb	0 → 1	0 → 1
transport-cc	0 → 1	0 → 1

answer는 협상된 코덱 하나(+RTX)만 광고하므로 offer처럼 수십 줄이 되지 않는 것이 정상이에요.

확인 방법

실제 ICE·DTLS 핸드셰이크를 맺고 클라이언트가 협상 후 적용하는 코덱 파라미터를 대조했어요. pion은 NACK·PLI·transport-cc 인터셉터 구동 여부를 SDP 텍스트가 아니라 이 값으로 결정해요.

	수정 전	수정 후
Receiver().GetParameters() 피드백	[]	ccm fir, goog-remb, nack, nack pli, transport-cc
Sender().GetParameters() 피드백	[]	동일
ICE/DTLS 연결	성공	성공

수정 전에도 연결 자체는 정상이고 피드백만 비어요 — "연결 실패"가 아니라 "초반 화질 깨짐 후 자가복구"였던 증상과 맞물려요.

go test ./internal/session -run 'RTCPFeedback' -v
go test ./internal/session ./internal/media -count=1

• TestAnswerRetainsOfferedRTCPFeedback (answer SDP 레벨) — 수정 전 RED → 수정 후 PASS
• TestConnectedPeerNegotiatesRTCPFeedback (연결 후 런타임 파라미터) — 수정 전 RED → 수정 후 PASS
• 두 패키지 모두 ok, -count=5 반복에서도 안정적이며 패키지 실행시간 변화 없음 (0.59s)
• gofmt -l internal/session 빈 출력, go vet ./internal/session 통과

검증하지 못한 것

• 미디어 실제 흐름은 미확인이에요. 통합 테스트의 더미 H.264 바이트가 디코딩 불가라 egress 파이프라인이 프레임을 내지 않아요. 검증 범위는 협상 계층까지예요.
• 브라우저 E2E는 못 했어요. cmd/server 빌드가 otel로 막혀 있는데, 원인은 저장소가 아니라 로컬 모듈 캐시 손상이에요 (go mod verify가 x/sys·x/text·gonum·protobuf 등 다수에 "dir has been modified"). 별도로 다뤄야 해요.

go build ./...는 otel 모듈 캐시 문제(semconv/v1.37.0: undefined: ErrorTypeOther)로 실패하는데, 이번 변경과 무관한 기존 이슈예요.

범위 밖

• drainRTCP의 PLI/FIR 분기 처리(2순위) — 현재 egress 인코더는 stdin 기반 별도 ffmpeg 프로세스라 실행 중 온디맨드 IDR을 강제할 채널이 없고, 인코더 재기동이냐 인프로세스 교체냐는 미결 설계 결정이에요. 별도 이슈로 다뤄요.
• GOP 단축 — 미리보기 레그 IDR 주기는 실측 1.000초로 이미 짧아요.
• YouTube RTMP egress([FEAT] 유튜브 송출 RTMP 구현 #29) 영향 여부 — 별도 확인 항목이에요.

문서

docs/가 .gitignore에 등록되어 있어 문서 갱신은 이 PR에 포함되지 않아요. 로컬에서 아래 두 문서를 갱신해 뒀어요.

• docs/operations/TROUBLESHOOTING.md — "측정 1" 표에 수정 후 값 컬럼 추가, "해결책" 1순위를 완료(fix: WebRTC answer SDP가 offer의 rtcp-fb를 누락해 송출 초반 화면이 깨짐 #90)로 표시
• docs/operations/streaming-startup-corruption-verification.md — 상단에 "#90에서 수정됨" 한 줄 추가(조사 시점 수치는 그대로)

Closes #90

🤖 Generated with Claude Code