Skip to content

Use rustls instead of openssl #133

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

Merged
merged 1 commit into from
Sep 29, 2022
Merged

Use rustls instead of openssl #133

merged 1 commit into from
Sep 29, 2022

Conversation

trueleo
Copy link
Contributor

@trueleo trueleo commented Sep 29, 2022

Fixes #121

Description

Use rustls instead of openssl. This involves switching out openssl feature for rustls in actix web. This commit introduces no addtional breaking changes in how keys are registered with actix_web, so they should work as before.

This PR has:

  • been tested to ensure log ingestion and log query works.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

Use rustls instead of openssl. This involves switching out openssl feature for
rustls in actix web. This commit introduces no addtional breaking changes in how
keys are registered with actix, so they should work as before.
@trueleo trueleo changed the title Use rustls Use rustls instead of openssl Sep 29, 2022
@trueleo trueleo marked this pull request as ready for review September 29, 2022 10:07
@nitisht nitisht merged commit a0196b4 into parseablehq:main Sep 29, 2022
@trueleo trueleo deleted the rustls branch October 1, 2022 06:33
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

Successfully merging this pull request may close these issues.

Released build is not portable
2 participants