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.

Sign up for GitHub

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

Jump to bottom

Additional Subscribe Errors #347

Open
martinduke opened this issue Jan 25, 2024 · 3 comments
Open

Additional Subscribe Errors #347

martinduke opened this issue Jan 25, 2024 · 3 comments
Assignees
@englishm
Labels
Wire Format Related to how messages are serialized and parsed

Comments

@martinduke
Copy link
Contributor

I woudld think that "track doesn't exist" and "unauthorized" would be useful error codes, but they don't seem to fit in 'Invalid Range' or 'Retry Track Alias'.
@afrind afrind added the Wire Format Related to how messages are serialized and parsed label Jan 31, 2024
@ianswett
Copy link
Collaborator

As discussed today, it'd be nice to have one indicating "Timeout".

@martinduke
Copy link
Contributor Author

Assigned to @kixelated as C-in-C of error codes.

@martinduke martinduke assigned englishm and unassigned kixelated Aug 14, 2024
englishm added a commit to englishm/moq-transport that referenced this issue Sep 18, 2024
@englishm
Add additional subscribe errors
cd500bd 
Adds threw new error codes:

- Track Does Not Exist (0x3)
- Unauthorized (0x4)
- Timeout (0x5)

As discussed in moq-wg#347
@englishm englishm mentioned this issue Sep 18, 2024
Merged
englishm added a commit to englishm/moq-transport that referenced this issue Sep 18, 2024
@englishm
Add additional subscribe errors
b122555 
Adds three new error codes:

- Track Does Not Exist (0x3)
- Unauthorized (0x4)
- Timeout (0x5)

As discussed in moq-wg#347
ianswett added a commit that referenced this issue Sep 18, 2024
@ianswett
Add additional subscribe errors (#529)
46c48ba 
Adds three new error codes:

- Track Does Not Exist (0x3)
- Unauthorized (0x4)
- Timeout (0x5)

As discussed in #347
@englishm englishm mentioned this issue Sep 18, 2024
Open
@ianswett
Copy link
Collaborator

Fixed by #529

@ianswett ianswett removed the Needs PR label Sep 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@englishm englishm

Labels
Wire Format Related to how messages are serialized and parsed
Projects
MoQT Tracking Board
Status: Needs PR
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kixelated @englishm @afrind @ianswett @martinduke