Skip to content

docs: document tokenId-scoped restream dedup behavior#705

Open
devin-ai-integration[bot] wants to merge 3 commits into
mainfrom
devin/1782192771-doc-restream-tokenid-dedup
Open

docs: document tokenId-scoped restream dedup behavior#705
devin-ai-integration[bot] wants to merge 3 commits into
mainfrom
devin/1782192771-doc-restream-tokenid-dedup

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a new "Re-streaming scope per publish token" section to millicast/distribution/re-streaming.mdx documenting that re-streaming is scoped per publish token — only one active ingest per token will be re-streamed at a time. Recommends using separate publish tokens for independent re-streams, or &norestream when multiple ingests must share a token.

Related: millicast/media-server#534 (code comments in Restreamer.js)

Link to Devin session: https://dolby.devinenterprise.com/sessions/3581ac14a78b4b269c43b4be6af6f620
Requested by: @ZitaLiao

Co-Authored-By: zita.liao <zita.liao@dolby.com>
@devin-ai-integration devin-ai-integration Bot requested a review from a team as a code owner June 23, 2026 05:33
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://THEOplayer.github.io/documentation/pr-preview/pr-705/

Built to branch gh-pages at 2026-06-23 07:33 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Comment thread millicast/distribution/re-streaming.mdx Outdated

To disable re-streaming on specific feeds add the **norestream** boolean [publishing parameter](/millicast/broadcast/publishing-parameters.md) to your configured publishing URLs. This is used to indicate which sources specifically should be used or not. For example, in a Multi-bitrate (MBR) configuration where there are multiple quality layers being ingested, you might re-stream only the highest quality layer and then add `&norestream` to any additional contribution layers. Similarly, if you have an interactive [multi-view](/millicast/playback/multi-view.md) broadcast where multiple sources are sharing the same publishing token, you can configure the URLs to only re-stream your intended source.

:::info Token-scoped dedup behavior

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  1. make another ## tile with something like "Restrict per publish token scope"
  2. "publisher node" is a bit too internal for the user. Find a better way for it, and hide tech details to just mention that if user would like to restream different ingest, better to use different token to each ingest, or explicitly use norestream to disable the ingest to restream with a same publish token

…l terminology

Co-Authored-By: zita.liao <zita.liao@dolby.com>
Comment thread millicast/distribution/re-streaming.mdx Outdated

## Re-streaming scope per publish token

Re-streaming is scoped to the **publish token**. Only one active ingest per publish token will be re-streamed at a time. If a new ingest starts using the same publish token while another is already re-streaming, the existing re-stream is stopped and the new ingest takes over.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

if it happens to route to different publish node, then it would not stop. So here, change "the existing re-stream is stopped" to "would be stopped". Let user knows restream with a same publish token is not suggest

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Addressed — changed "is stopped" → "would be stopped" and added a bold recommendation that re-streaming multiple ingests with the same publish token is not recommended, noting the behavior may be unpredictable.

Co-Authored-By: zita.liao <zita.liao@dolby.com>

## Re-streaming scope per publish token

Re-streaming is scoped to the **publish token**. Only one active ingest per publish token will be re-streamed at a time. If a new ingest starts using the same publish token while another is already re-streaming, the existing re-stream would be stopped and replaced by the new ingest. Because of this, **re-streaming multiple ingests with the same publish token is not recommended** — the behavior may be unpredictable.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

"Only one active ingest per publish token will be re-streamed at a time. If a new ingest starts using the same publish token while another is already re-streaming, the existing re-stream would be stopped and replaced by the new ingest."
Change to (please polish)
"Only one active ingest per publish token per publish node will be re-streamed at a time. If a new ingest starts using the same publish token route to (decided by load balance strategy) a same publish node with another is already re-streaming, the existing re-stream would be stopped and replaced by the new ingest. So multiple stream with a same public token would randomly fail. In general, restream different streams with a same publish token is not full-function supported and so not suggest."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant