docker-mautic icon indicating copy to clipboard operation
docker-mautic copied to clipboard

On-the-fly asset generation is broken

Open TonyBogdanov opened this issue 1 year ago • 11 comments
trafficstars

When running the docker image of mautic5, the asset generation of media files fails for a lot of JS files and outputs their paths instead of their contents.

This issue was originally posted here: https://github.com/mautic/mautic/issues/13076, but I think this is the proper place for it.

The cause is this: https://github.com/mautic/docker-mautic/blob/084bcee5cc963379a8c519fe2fad4df8eeb37615/apache/Dockerfile#L62

All NPM packages are forcefully removed (apart from jquery and vimeo-froogaloop2) for some reason, but that breaks the on-the-fly generation of media/js/libraries.js for example.

TonyBogdanov avatar May 02 '24 10:05 TonyBogdanov

Thanks for flagging up the issue @TonyBogdanov - will share it in the Docker channel on Slack and hopefully someone can jump in and take a look!

RCheesley avatar May 02 '24 12:05 RCheesley

I have same problem using version image 5.1.0 from Docker Hub.

image

miguelurtado avatar Jun 26 '24 02:06 miguelurtado

@mollux was there a reason for this line?: https://github.com/mautic/docker-mautic/blob/mautic5/fpm/Dockerfile#L62 It seems like it's causing a bunch of issues.

escopecz avatar Jun 27 '24 13:06 escopecz

Removing that line fixes the issue.

tathougies avatar Sep 24 '24 01:09 tathougies

Hey there! I'm helping the new maintainers of this repository. It was neglected for some time but that changes now! Could you please re-test if you are still getting this issue so we could focus our limited time on the issues that are still relevant? If there will be no response we'll close this issue in 1 week. It can always be re-opened later on.

IonutOjicaDE avatar Apr 15 '25 16:04 IonutOjicaDE

Hi @IonutOjicaDE, the latest commit in this repo was on Apr 16, 2024 and I found / opened this issue on May 2, so it should most definitely still exist. Note, this is not an issue with Mautic, but the way the docker images are generated (Dockerfile).

TonyBogdanov avatar Apr 15 '25 18:04 TonyBogdanov

Hey @TonyBogdanov ,

Thank you for the clarification. We'll take a look on it once the core team gives us permissions to perform actions on the repository!

cibero42 avatar Apr 17 '25 13:04 cibero42

@IntrovertedFL

Can you please have a look of this issue?

Sorry, but I couldn't assign it to you for some reason.

cibero42 avatar Apr 25 '25 07:04 cibero42

Hey @O-Mutt ,

Could you please check if this issue still holds for v5.2.5?

Aaron is unavailable to check it, hence why I'm asking for your help.

Thanks!

cibero42 avatar May 04 '25 11:05 cibero42

Yeah, I can take a look in the morning tomorrow 👍👌

O-Mutt avatar May 04 '25 12:05 O-Mutt

Thanks :)

cibero42 avatar May 04 '25 12:05 cibero42