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

[core]Fixing the group option that should be taken from a socket #2912

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ethouris
Copy link
Collaborator

@ethouris ethouris commented Mar 11, 2024

Changed: Added a possibility to return the option value that was set on the socket, if there is at least one member.

The problem: this should rather concern only selected types of options, not all options. So far probably it should only
apply to the following options:

  • SRTO_RCVLATENCY, SRTO_PEERLATENCY, SRTO_LATENCY
  • SRTO_CRYPTOMODE

Without this the test will fail, and it's also not expected. We need this to work only for those options that could be altered as a result of the connection and involved parameter exchange.

Addresses #2918.

@maxsharabayko maxsharabayko added this to the v1.5.4 milestone Mar 13, 2024
@maxsharabayko maxsharabayko added Type: Bug Indicates an unexpected problem or unintended behavior [core] Area: Changes in SRT library core labels Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[core] Area: Changes in SRT library core Type: Bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants