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

[BUG] Setting Socket Options not Allowed on a Group Succeed #3074

Open
1 of 5 tasks
maxsharabayko opened this issue Nov 14, 2024 · 0 comments
Open
1 of 5 tasks

[BUG] Setting Socket Options not Allowed on a Group Succeed #3074

maxsharabayko opened this issue Nov 14, 2024 · 0 comments
Labels
[API] Area: Changes in SRT library API Type: Bug Indicates an unexpected problem or unintended behavior
Milestone

Comments

@maxsharabayko
Copy link
Collaborator

maxsharabayko commented Nov 14, 2024

TODO:

  • Make a short list of group socket options that are not allowed to be set/get on a group.
  • Are there options that are allowed but should not be?
  • Fix Documentation, e.g. setting SRTO_BINDTODEVICE on a group does not make sense.
  • Improve unit tests of group socket options.

BUGS:

According to the API-socket-options.md document the following options are not supported by a group:

However, at least setting SRTO_RENDEZVOUS to true returns SUCCESS and even tries to establish a rendezvous connection. Looking at the code the rest can also be set on a group without any error.

Can't read on a group:

Can be set on a group socket, but documented as the opposite

See PR #3086 with unit tests extended to group socket options.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[API] Area: Changes in SRT library API Type: Bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

1 participant