Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

kTLS support #49

Merged
merged 30 commits into from
Sep 12, 2023
Merged

kTLS support #49

merged 30 commits into from
Sep 12, 2023

Conversation

bonifaido
Copy link
Member

@bonifaido bonifaido commented Sep 1, 2023

Description

Currently only the TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305_SHA256 cipher is implemented, this is the default cipher of BearSSL.

Fixes #54

Contains a lot of (untracked) other fixes and a small test that verifies that sendfile works with kTLS (via NGiNX).

TODO:

  • add test for app without kTLS (I have added a test, but it works locally only, somehow not on GH Actions)
  • add test with nginx (sendfile should be stressed)

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@bonifaido bonifaido changed the title [WIP] kTLS kTLS support Sep 7, 2023
@bonifaido bonifaido marked this pull request as ready for review September 7, 2023 13:42
@bonifaido bonifaido requested a review from a team September 7, 2023 13:42
@bonifaido bonifaido merged commit 41762be into main Sep 12, 2023
1 check passed
@bonifaido bonifaido deleted the kTLS branch September 12, 2023 13:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] kTLS sendfile can't be received on client side
2 participants