search

docker-library / drupal

Docker Official Image packaging for Drupal
280 stars 203 forks source link

New to drupal, first time install all good but there are some missing services drupal is complaining about? #270

Open kelvtech-co-uk opened 3 weeks ago

kelvtech-co-uk commented 3 weeks ago

Hi, I'm checking in that I'm not creating more of a problem for down the road that I don't yet understand. Anyway the docker official image for drupal was super convienent to find and setup so thank you for that. I noted that on first install drupal's status report was advising a few things:-

Without really knowing why this is the case I found my way to this repo, made a few small modications and now have a custom image which addresses the above. Other than the additional time to build the image it looks to be only +149Mb in size.

Is there a problem I'm meandering over here, would appreicate any quick pointers to read further. Im migrating my partners small baking website, happy to send brownies in exchange for help :smiley:

mstenta commented 3 weeks ago
  • output buffering isn't enabled

See #256

  • PHP APCu isn't enabled yet it can be advantages for performance gains
  • Upload progress is possible but also not enabled.

Not sure if these have been discussed but if you have made changes feel free to open a PR for consideration!

tianon commented 3 weeks ago

Having pointers to explicit upstream Drupal recommendations for enabling more functionality by default will help make a stronger case for us doing so :eyes:

kelvtech-co-uk commented 3 weeks ago

Thanks for the feedback the PR is here. I found a drupal note regards the Upload Progress status report item being out of date and to ignore it. Would this qualify as something to note in the /docker-library/docs or too small beer to bother?