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

a small bug when use quic_client test connection migration with fixed client_connection_id_length #62

Open
bixuelantian opened this issue Dec 18, 2023 · 0 comments

Comments

@bixuelantian
Copy link

bixuelantian commented Dec 18, 2023

@bnc-google @wu-bin @vasilvv hi,when I use quic_client test connection migration with fixed client_connection_id_length=16,I find a smalle bug:

In the quic_client, when you specify a fixed client_connection_id_length, will call the function QuicConnection::set_client_connection_id。then the client_connection_id_length will be used to call SetExpectedClientConnectionIdLength to set expected_client_connection_id_length_, which is used to decide dst id length when a packet comes in.

On the other hand, the client issued client id with a default fixed length is 8,when connection migration happens, the server will choose an unused client id with length is 8 to send packet to client。 so when then expected_client_connection_id_length_ is not 8, the client will get a wrong length dst id.

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

No branches or pull requests

1 participant