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

Ensure no data is sent after a stream reset #444

Merged
merged 2 commits into from
Jan 7, 2024

Conversation

jlaine
Copy link
Contributor

@jlaine jlaine commented Jan 7, 2024

Once a RESET has been requested on a stream, the stream's state is exclusively determined by the RESET being acknowledged. Ensure that we never send out any more data after a RESET being acknowledged, even if data is lost.

jlaine added 2 commits January 7, 2024 14:02
Once a RESET has been requested on a stream, the stream's state is
exclusively determined by the RESET being acknowledged. Ensure that we
never send out any more data after a RESET being acknowledged, even if
data is lost.
Copy link

codecov bot commented Jan 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (0c38321) 100.00% compared to head (83dab68) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #444   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           25        25           
  Lines         4872      4875    +3     
=========================================
+ Hits          4872      4875    +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jlaine jlaine merged commit 70c2fa6 into aiortc:main Jan 7, 2024
29 checks passed
@jlaine jlaine deleted the no-data-after-reset branch January 7, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant