You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
WHAT AND WHY
Update the generated Dockerfile to use an upstream image, similar to how the current laravel-docker repository is..
There are three primary reasons behind this move:
To remove the scope of having to maintain the custom image built from the laravel-docker repository for SOC2 compliance. This was brought up from a customer request for periodic security patches and maintenance for the custom image we have provided, which we currently cannot fully commit to.
Consistency: The generated Dockerfiles for other frameworks like Rails, Elixir, Javascript, all use upstream images as base. For consistency, we should also follow for the Dockerfile generated for Laravel apps.
Finally, this move will result in depositing config files to the users' project directory where they can visibly and readily find them, providing them full control over these files, as I quote @rubys :
they can rerun dockerfile-laravel when they want to pick up changes, hold back, and even make their own changes to these files."
HOW:
It's simple really! The laravel-docker repository already makes use of a Dockerfile that makes use of an upstream base image. So, we'll just make use of that Dockerfile. Further, since that Dockerfile requires several config file, we'll copy those over to the dockerfile-laravel repository as well.
Basically copy over the laravel-docker logic to dockerfile-laravel. This will result in the same Dockerfile logic, while using an upstream base image instead of the custom one we've provided.
To make this a clean move and reduce the number of files to check. I'll create more than one PR to address this issue:
a. Move the config files required by the laravel-dockerfile over to this repository
b. Move the Dockerfile logic of laravel-dockerfile over to this repository
The text was updated successfully, but these errors were encountered:
WHAT AND WHY
Update the generated Dockerfile to use an upstream image, similar to how the current laravel-docker repository is..
There are three primary reasons behind this move:
HOW:
It's simple really! The laravel-docker repository already makes use of a Dockerfile that makes use of an upstream base image. So, we'll just make use of that Dockerfile. Further, since that Dockerfile requires several config file, we'll copy those over to the dockerfile-laravel repository as well.
Basically copy over the laravel-docker logic to dockerfile-laravel. This will result in the same Dockerfile logic, while using an upstream base image instead of the custom one we've provided.
NOTES:
a. Move the config files required by the laravel-dockerfile over to this repository
b. Move the Dockerfile logic of laravel-dockerfile over to this repository
The text was updated successfully, but these errors were encountered: