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

[!] fix test error on ubuntu as retransmission might happen and the rcvd/sent bytes will be more than specified #353

Merged
merged 4 commits into from
Oct 27, 2023

Conversation

Kulsk
Copy link
Collaborator

@Kulsk Kulsk commented Oct 26, 2023

if packet is lost or retransmit on PTO, the final rcvd/sent bytes will exceed the specified amount, makes the test case fail

@Kulsk Kulsk merged commit 0ff85e0 into alibaba:main Oct 27, 2023
6 of 8 checks passed
yangfurong pushed a commit that referenced this pull request Nov 2, 2023
* [+] support multi-path 06 in transport parameter

* [+] add new frame types for 06

* [+] add generation for 06 frames

* [+] add sending for path standby and available

* [+] add binding path standby/available with path_challenge

* [+] error handling

* [+] add support in version check

* [+] 添加解析入口

* [+] add mp_ack support for 06

* [+] add support for mp ack in 06

* [!] fix test error on ubuntu as retransmission might happen and the rcvd/sent bytes will be more than specified (#353)

* [!] fix error on timer_based_dgram_probe testcase and timer_based_dgram_probe

* [~] fix available flag setting

* [+] support multi-path 06 in transport parameter

* [+] add new frame types for 06

* [+] add generation for 06 frames

* [+] add sending for path standby and available

* [+] add binding path standby/available with path_challenge

* [+] error handling

* [+] add support in version check

* [+] 添加解析入口

* [+] add mp_ack support for 06

* [+] add support for mp ack in 06

* [~] fix available flag setting

* [+] demo client send path available after handshake finished

* [+] add log

---------

Co-authored-by: Wu Zhao <zhaowu.zw@alibaba-inc.com>
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.

2 participants