Skip to content

Remove openssl and pq #154

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 4 commits into from
May 20, 2025
Merged

Remove openssl and pq #154

merged 4 commits into from
May 20, 2025

Conversation

clux
Copy link
Owner

@clux clux commented May 20, 2025

Both EOL for ages, not something I use anymore, and has maintained alternatives (in form of the blackdex image for pq, and rustls for openssl).
Having these libraries in here creates a false confidence that these are OK to use (at the given versions) when they are not. Better to link to more specific variants that care about these libraries.

Closes #153
Closes #81

This also saves quite a bit on the arm build. 65m to 35m (most of which is tests now so can be optimized further).

Minor test specifics;

  • updates test dependency versions to fix fake dependency vulns
  • shift rustls / hyper test to a new version where openssl is no longer default
  • remove diesel test using pq

clux added 3 commits May 20, 2025 21:52
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux changed the title Remove openssl and pq Remove openssl and pq May 20, 2025
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux marked this pull request as ready for review May 20, 2025 22:01
@clux clux merged commit 80f4e22 into main May 20, 2025
6 checks passed
@clux clux deleted the rm-openssl-pq branch May 20, 2025 22:34
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.

remove openssl libpq >= 12 fails to static link with openssl
1 participant