Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

moq-pub: Avoid namespace conflict #65

Closed
englishm opened this issue Sep 6, 2023 · 3 comments
Closed

moq-pub: Avoid namespace conflict #65

englishm opened this issue Sep 6, 2023 · 3 comments
Labels
good first issue Good for newcomers

Comments

@englishm
Copy link
Collaborator

englishm commented Sep 6, 2023

We should either (or both):

  • Allow users to specify a particular namespace as a command line argument
  • Generate UUIDs as part of the namespaces (or maybe just as the default if nothing is specified) to avoid conflict
@englishm englishm added the good first issue Good for newcomers label Sep 6, 2023
@englishm
Copy link
Collaborator Author

englishm commented Sep 6, 2023

The current value is simply hardcoded here:

media_runner.announce("quic.video/moq-pub-foo", media.source()).await?;

@zafergurel
Copy link
Contributor

I got this.

@englishm
Copy link
Collaborator Author

englishm commented Sep 7, 2023

Resolved by #66. Thanks @zafergurel!

@englishm englishm closed this as completed Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants