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

Synchronize changes from api-platform template (aka switch to FrankenPHP) #4453

Open
usu opened this issue Jan 14, 2024 · 1 comment · May be fixed by #5123
Open

Synchronize changes from api-platform template (aka switch to FrankenPHP) #4453

usu opened this issue Jan 14, 2024 · 1 comment · May be fixed by #5123
Assignees

Comments

@usu
Copy link
Member

usu commented Jan 14, 2024

The last PRs for synchronizing changes form the official api-platform were approx. 1 year ago:
#3304
#3255

In the meantime, the following changes were added:
api-platform/api-platform@7642f86...main

The biggest change is the move from php-fpm to frankenphp, which came out beta last month (https://laravel-news.com/frankenphp).

Do we plan to switch to FrankenPHP as well? Or do we judge this as too early?

The biggest benefits I'd see:

  • PHP and Caddy in 1 container instead of 2 separate ones
  • Potential performance benefits by running Symfony in worker mode (similar like node; boot once and then serve all incoming requests). Of course, performance improvements would need to be tested and verified first.

Remaining:

  • upgrade to Postgres 16
  • FrankenPHP runtime mode
@usu usu added the Meeting Discuss Am nächsten Core-Meeting besprechen label Jan 14, 2024
@manuelmeister
Copy link
Member

Core Meeting Discuss

  • Cool idea. Local dev would probably work with one container as well.
  • If someone want to tackle this, we are not against it.

@BacLuc BacLuc removed the Meeting Discuss Am nächsten Core-Meeting besprechen label Mar 6, 2024
@usu usu self-assigned this May 4, 2024
@usu usu linked a pull request May 4, 2024 that will close this issue
3 tasks
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 a pull request may close this issue.

3 participants