Skip to content

sctp fixes - #479

Merged
jmillan merged 1 commit into
v3from
sctp-fix
Nov 11, 2020
Merged

jmillan merged 1 commit into
v3from
sctp-fix

Conversation

@jmillan

@jmillan jmillan commented Nov 11, 2020

Copy link
Copy Markdown
Member
  • DataConsumer: make 'setBufferedAmountLowThreshold' trigger 'bufferedamountlow'
    if the buffered amount data is less or same than the given threshold.

  • SctpAssociation: Increase buffered amount data before sending it in
    order to avoid inconsistencies if 'onSendSctpData' fires while 'usrsctp_senv'
    is being executed. Be conservative.

- DataConsumer: make 'setBufferedAmountLowThreshold' trigger 'bufferedamountlow'
    if the buffered amount data is less or same than the given threshold.
- SctpAssociation: Increase buffered amount data before sending it in
  order to avoid inconsistencies if 'onSendSctpData' fires while 'usrsctp_senv'
  is being executed. Be conservative.
@jmillan
jmillan requested a review from ibc November 11, 2020 13:56
@jmillan

jmillan commented Nov 11, 2020

Copy link
Copy Markdown
Member Author

We don't have full control on the real buffered data in usrsctp. With this changes we can subscribe to 'bufferedamountlow' event and set the buffered amount low threshold and we will be sure that:

  • the event will fire if there is less or same buffered buffer as indicated.
  • otherwise the event will fire as soon as there is less or same buffered buffer as indicated.

Comment thread worker/src/RTC/DataConsumer.cpp
@jmillan
jmillan merged commit dcb9c1b into v3 Nov 11, 2020
@jmillan
jmillan deleted the sctp-fix branch November 11, 2020 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants