search

moq-wg / moq-transport

draft-ietf-moq-transport
Other
87 stars 22 forks source link

Expand Type field in control messages #541

Closed LPardue closed 2 months ago

LPardue commented 2 months ago

Fixes https://github.com/moq-wg/moq-transport/issues/537

suhasHere commented 2 months ago

Since a control message is defined as 

MOQT Control Message {
  Message Type (i),
  Message Payload (b),
}

I think we also need explicit length field for the entire message. May be we should do both the changes together ??

LPardue commented 2 months ago

Yes! I didn't realise that until I started on https://github.com/moq-wg/moq-transport/pull/544. And I did the work there. Whatever is the easiest to merge first, I can rebate or respin the PRs so we have full expansion in the next draft release

suhasHere commented 2 months ago

@LPardue yeah .. That was one of the reasons made be think that we should combine this and #544 as on pass on control message

LPardue commented 2 months ago

closing in favor of the mega PR that people asked for: https://github.com/moq-wg/moq-transport/pull/550