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

tests: net: sockets: tls: Increase connect timeout #79003

Conversation

rlubos
Copy link
Contributor

@rlubos rlubos commented Sep 25, 2024

Since fc007ee TLS sockets support timeout based on CONFIG_NET_SOCKETS_CONNECT_TIMEOUT value. The TLS test suite reduced this config value to speed up the tests, however it turns out the 100ms timeout is too low to finalize the handshake on slower platforms (nrf52840 specifically). Therefore increase the timeout value to 200ms to prevent test failures on those.

Since fc007ee TLS sockets support
timeout based on CONFIG_NET_SOCKETS_CONNECT_TIMEOUT value. The TLS test
suite reduced this config value to speed up the tests, however it turns
out the 100ms timeout is too low to finalize the handshake on slower
platforms (nrf52840 specifically). Therefore increase the timeout value
to 200ms to prevent test failures on those.

Signed-off-by: Robert Lubos <[email protected]>
@zephyrbot zephyrbot added area: Networking area: Sockets Networking sockets size: XS A PR changing only a single line of code labels Sep 25, 2024
@mmahadevan108 mmahadevan108 merged commit 7aef84c into zephyrproject-rtos:main Sep 28, 2024
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Networking area: Sockets Networking sockets size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants