Remove unused openssl headers - Githubissues

nodejs / docker-node

Official Docker Image for Node.js :whale: :turtle: :rocket:

https://hub.docker.com/_/node/

MIT License

8.24k stars 1.97k forks source link

Remove unused openssl headers #1996

Closed yehonatanz closed 10 months ago

yehonatanz commented 10 months ago

Remove unused OpenSSL headers for architectures other than current one - now for alpine and for node21. (follow up of #1923 )

Description

See #1923

Motivation and Context

Reducing alpine image sizes by 48MB:

Testing Details

Installed bcrypt and hashed a random string.

Types of changes

[ ] Documentation
[ ] Version change (Update, remove or add more Node.js versions)
[ ] Variant change (Update, remove or add more variants, or versions of variants)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[X] Other (none of the above)

Checklist

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING.md document.
[X] All new and existing tests passed.

github-actions[bot] commented 10 months ago

Created PR on the official-images repo (https://github.com/docker-library/official-images/pull/15729). See https://github.com/docker-library/faq#an-images-source-changed-in-git-now-what if you are wondering when it will be available on the Docker Hub.