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

FrankenPHP variation #283

Closed
wants to merge 2 commits into from
Closed

Conversation

sneycampos
Copy link

This PR adds FrankenPHP as a new variation to run Roundcube

I'm personally using and looks everything working.

There's some necessary updates in the README for frankenphp variation, like how to auto generate TLS certificates, etc.

@@ -0,0 +1,96 @@
FROM dunglas/frankenphp:1.3.1-php8.2-bookworm
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

running as root user, let me know if we may create and run as a non root user

@pabzm
Copy link
Member

pabzm commented Nov 25, 2024

Thank you for the idea! Could you explain why you think that we should publish this additional image as an official one?

I'm asking because I'm not sure if I want to do that. Haven't got any personal experience with it and don't know anyone who does. And if we start to support it we would need to carry it long-term, which could become a burden if no-one here knows it well.

Do you know more people that would like to see such an image?

@thomascube
Copy link
Member

I share @pabzm's concerns. We better stick with the official PHP distributions and limit the variants to a minimum. If alternative webservers want to be used we recommend to use the FPM variant along the a standalone webserver image like nginx or caddy.

@sneycampos sneycampos closed this Nov 28, 2024
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.

3 participants