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

gnutls, improve shutdown #13790

Closed
wants to merge 1 commit into from
Closed

Conversation

icing
Copy link
Contributor

@icing icing commented May 27, 2024

local ftp upload tests sometimes failed with an invalid TLS record being reported by gnutls. vsftp did log that the shutdown was not regarded as clean, failing the control connection thereafter.

These changes make test_31_05 work reliable locally.

  • on closing the SSL filter, shutdown READ and WRITE
  • on closing, try a receive after shutdown is sent
  • convert to DEBUGF to CURL_TRC_CF

local ftp upload tests sometimes failed with an invalid
TLS record being reported by gnutls. vsftp did log that
the shutdown was not regarded as clean, failing the
control connection thereafter.

These changes make test_31_05 work reliable locally.

- on closing the SSL filter, shutdown READ *and* WRITE
- on closing, try a receive after shutdown is sent
- convert to DEBUGF to CURL_TRC_CF
@github-actions github-actions bot added the TLS label May 27, 2024
@bagder bagder closed this in 7bbad0c May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

2 participants