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

[Bug]: test_client need remove test_session(file) when changing host of url #67

Closed
nandsky opened this issue Jan 22, 2022 · 1 comment · Fixed by #82
Closed

[Bug]: test_client need remove test_session(file) when changing host of url #67

nandsky opened this issue Jan 22, 2022 · 1 comment · Fixed by #82
Labels
🍰 good first issue Good for newcomers 🐞 bug Something isn't working

Comments

@nandsky
Copy link

nandsky commented Jan 22, 2022

What happened?

test_client need remove test_session(file) when changing host of url

Steps To Reproduce

Information and Steps to reproduce the behavior.

  1. ./test_client -u "https://aaa.dinoflare.net/1M" -G -a 11.158.253.151 -p 443 (success)
  2. ./test_client -u "https://bbb.dinoflare.net/1M" -G -a 11.158.253.151 -p 443 (failed)

Relevant log output

[2022/01/22 11:40:10 794833] [info] |xqc_tls_ctx_create|set cipher suites suc|ciphers:TLS_AES_128_GCM_SHA256:TLS_AES_256_GCM_SHA384:TLS_CHACHA20_POLY1305_SHA256
[2022/01/22 11:40:10 794899] [info] |xqc_tls_ctx_register_alpn|alpn registered|alpn:h3-29|alpn_list:^Eh3-29
[2022/01/22 11:40:10 794905] [info] |xqc_engine_add_alpn|alpn registered|alpn:h3-29|
[2022/01/22 11:40:10 794909] [info] |xqc_tls_ctx_register_alpn|alpn registered|alpn:h3|alpn_list:^Eh3-29^Bh3
[2022/01/22 11:40:10 794912] [info] |xqc_engine_add_alpn|alpn registered|alpn:h3|
[2022/01/22 11:40:10 794918] [info] |xqc_tls_ctx_register_alpn|alpn registered|alpn:transport|alpn_list:^Eh3-29^Bh3 transport
[2022/01/22 11:40:10 794921] [info] |xqc_engine_add_alpn|alpn registered|alpn:transport|
[2022/01/22 11:40:10 795417] [debug] |scid:f9f11c4e926737a5|xqc_send_ctl_timer_set|type:IDLE|expire:1642822820795411|now:1642822810795411|interv:10000000|
[2022/01/22 11:40:10 795441] [debug] |scid:f9f11c4e926737a5|xqc_conn_create|success|scid:f9f11c4e926737a5|dcid:aea89fff4f4185d8|conn:00000000012E7110|
[2022/01/22 11:40:10 795447] [debug] |scid:f9f11c4e926737a5|xqc_create_crypto_stream|encrypt_level:0|cur_state:C_INIT|
[2022/01/22 11:40:10 795935] [error] |scid:f9f11c4e926737a5|xqc_tls_send_alert|ssl alert|level:0|alert:80|error:error:142120E7:SSL routines:tls_construct_ctos_early_data:inconsistent early data sni
[2022/01/22 11:40:10 795948] [error] |scid:f9f11c4e926737a5|xqc_conn_tls_error_cb|tls error|0x50|
[2022/01/22 11:40:10 795953] [error] |scid:f9f11c4e926737a5|xqc_conn_tls_error_cb|conn:00000000012E7110|err:0x150|addr or cid not avail|
[2022/01/22 11:40:10 795960] [error] |scid:f9f11c4e926737a5|xqc_tls_do_handshake|TLS handshake error:error:1424A044:SSL routines:write_state_machine:internal error|
[2022/01/22 11:40:10 795964] [error] |scid:f9f11c4e926737a5|xqc_client_create_tls|init tls error
[2022/01/22 11:40:10 795967] [error] |scid:f9f11c4e926737a5|xqc_conn_destroy|in XQC_CONN_FLAG_TICKING|00000000012E7110|
[2022/01/22 11:40:10 795970] [error] |xqc_client_connect|create connection error|
[2022/01/22 11:40:10 795973] [error] |xqc_h3_connect|xqc_client_connect error|
[2022/01/22 11:40:10 796011] [debug] |xqc_engine_destroy|begin|
[2022/01/22 11:40:10 796016] [report] |scid:f9f11c4e926737a5|xqc_conn_destroy|00000000012E7110|srtt:250000|retrans rate:0.0000|send_count:0|lost_count:0|tlp_count:0|spurious_loss_count:0|recv_count:0|has_0rtt:0|0rtt_accept:0|token_ok:-1|handshake_time:0|first_send_delay:0|conn_persist:606|err:0x150|addr or cid not avail|
[2022/01/22 11:40:10 796023] [debug] |scid:f9f11c4e926737a5|xqc_send_ctl_destroy|destroy|
@Kulsk
Copy link
Collaborator

Kulsk commented Jan 22, 2022

actually session ticket, transport parameter, token is related to destination QUIC servers, maybe it's better to link the stored contents with host.
by the way, changing ALPN will also lead to inconsistent ALPN error when using the session ticket.

@ruiqizhou ruiqizhou added 🐞 bug Something isn't working 🍰 good first issue Good for newcomers labels Jan 24, 2022
@Kulsk Kulsk linked a pull request Feb 11, 2022 that will close this issue
@Kulsk Kulsk closed this as completed in #82 Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍰 good first issue Good for newcomers 🐞 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants