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

Can you add openssl inside the container please ? #2

Closed
physispc opened this issue Sep 23, 2022 · 2 comments
Closed

Can you add openssl inside the container please ? #2

physispc opened this issue Sep 23, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@physispc
Copy link

Hi I need to use tls for harakamail.

Adding the 'tls' line inside 'config/plugins' and configuring 'config/tls.ini', the container doesn't start up.
Here is the error message :

[CRIT] [-] [core] Error: spawn openssl ENOENT
[CRIT] [-] [core]     at ChildProcess._handle.onexit (node:internal/child_process:283:19)
[CRIT] [-] [core]     at onErrorNT (node:internal/child_process:476:16)
[CRIT] [-] [core]     at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
[NOTICE] [-] [core] Shutting down

Adding the openssl package inside the container resolves this issue.

/ # apk add openssl
fetch https://dl-cdn.alpinelinux.org/alpine/v3.16/main/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/v3.16/community/x86_64/APKINDEX.tar.gz
(1/1) Installing openssl (1.1.1q-r0)
Executing busybox-1.35.0-r17.trigger
OK: 9 MiB in 18 packages
/ # openssl
OpenSSL> / #

Thanks !

@tyranron
Copy link
Member

@physispc would love to see a PR.

@tyranron tyranron added the enhancement New feature or request label Sep 26, 2022
@tyranron tyranron self-assigned this Oct 24, 2022
@tyranron
Copy link
Member

@physispc I've added APK_INSTALL_PACKAGES env var to pre-install system packages on startup, like we do with HARAKA_INSTALL_PLUGINS.

I decided to not hardcode openssl into the image, because other users may prefer alternatives like libretls, and in general case may require additional packages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants