Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 19 additions & 5 deletions draft-lcurley-moq-lite.md
Original file line number Diff line number Diff line change
Expand Up @@ -361,12 +361,17 @@ A subscriber sends an ANNOUNCE_PLEASE message to indicate it wants to receive an
ANNOUNCE_PLEASE Message {
Message Length (i)
Broadcast Path Prefix (s),
Exclude Hop (i),
}
~~~

**Broadcast Path Prefix**:
Indicate interest for any broadcasts with a path that starts with this prefix.

**Exclude Hop**:
If non-zero, the publisher SHOULD skip ANNOUNCE messages for broadcasts whose hop list contains this value.
This is used by relays to avoid routing loops in a cluster.

The publisher MUST respond with ANNOUNCE messages for any matching and active broadcasts, followed by ANNOUNCE messages for any future updates.
Implementations SHOULD consider reasonable limits on the number of matching broadcasts to prevent resource exhaustion.

Expand All @@ -384,7 +389,8 @@ ANNOUNCE Message {
Message Length (i)
Announce Status (i),
Broadcast Path Suffix (s),
Hops (i),
Hop Count (i),
Hop ID (i) ...,
}
~~~

Expand All @@ -397,10 +403,14 @@ A flag indicating the announce status.
**Broadcast Path Suffix**:
This is combined with the broadcast path prefix to form the full broadcast path.

**Hops**:
The number of hops from the origin publisher.
This is used as a tiebreaker when there are multiple paths to the same broadcast.
A relay SHOULD increment this value when forwarding an announcement.
**Hop Count**:
The number of Hop ID values that follow.

**Hop ID**:
A unique identifier for each relay in the path from the origin publisher.
Each relay SHOULD append its own Hop ID when forwarding an announcement.
The number of hops is used as a tiebreaker when there are multiple paths to the same broadcast.
A value of 0 indicates an unknown hop (e.g. when bridging from an older protocol version).
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated


## SUBSCRIBE
Expand Down Expand Up @@ -608,6 +618,10 @@ A generic library or relay MUST NOT inspect or modify the contents unless otherw

# Appendix A: Changelog

## moq-lite-04
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Fix heading style at Line 623 to satisfy markdownlint MD003.

## moq-lite-04 is ATX style, but the configured rule expects setext style. Please convert this heading to setext (or align rule config if ATX is intended globally).

🧰 Tools
🪛 markdownlint-cli2 (0.22.0)

[warning] 623-623: Heading style
Expected: setext; Actual: atx

(MD003, heading-style)

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@draft-lcurley-moq-lite.md` at line 623, The heading "moq-lite-04" currently
uses ATX style ("## moq-lite-04") which violates markdownlint MD003; replace the
ATX line with a setext-style heading by removing the leading "##" and adding a
setext underline on the following line (for an H2 use a line of three or more
hyphens, e.g. place "moq-lite-04" on one line and "---" on the next). Ensure no
extra trailing spaces so the new setext heading (moq-lite-04 + ---) replaces the
original ATX heading.

- ANNOUNCE `Hops` count replaced with explicit `Hop ID` list for loop detection.
- Added `Exclude Hop` to ANNOUNCE_PLEASE for relay loop avoidance.

## moq-lite-03
- Version negotiated via ALPN (`moq-lite-xx`) instead of SETUP messages.
- Removed Session, SessionCompat streams and SESSION_CLIENT/SESSION_SERVER/SESSION_UPDATE messages.
Expand Down
Loading