Docker: svg support for imagick

[fuse314]

Pull Request #575 has solved adding imagick manually to the container - thank you for this! - but imagick is missing SVG Support.
After changing my custom docker image back to the default (nextcloud:15-fpm-alpine), I am now stuck with the blue default favicons. My custom image showed the current app icon with the themed background.

Since I am new to this docker game, I cannot implement this change myself (I basically copy-pasted my code from the example docker files)

• What packages can I add manually to enable SVG support in imagick?
• What changes are needed in the official docker file to add support for SVG?
• Is SVG support something that should be added for everyone (i.E. official docker file) or should it be optional (i.E. add an example in the documentation) ?

[shoetten]

You need to add libmagickcore-6.q16-3-extra to your image. This will add support for additional formats, including svg, to imagemagick.

For example change the part in the example where all the apt-get installs happen, to include this package:

RUN set -ex; \
    \
    apt-get update; \
    apt-get install -y --no-install-recommends \
        ffmpeg \
        smbclient \
        supervisor \
        # svg support
        libmagickcore-6.q16-3-extra \
    ; \
    rm -rf /var/lib/apt/lists/*

[J0WI]

There's an ongoing discussion in https://github.com/nextcloud/server/issues/13099

[kesselb]

Do you think we could publish another flavor with svg support and notice the user about the security concerns?

[J0WI]

I think this does only affect Debian based images, so I filled #634 for consistency.

[J0WI]

Looks like this does actually require the imagemagick binary. I've added this to the full example in #634, but I think it's not worth adding this by default (nextcloud/server#13099). You can easily convert the favicon locally to a PNG.

[bkraul]

My point is that if it is "not worth" adding the package by default, why is the pecl extension being installed? As outlined, the extension shows loaded, but with no supported file formats. It used to be that neither the PECL extension or the imagemagick package were included. I have a derived image that I maintain with the added functionality here.

I used to have to add support for imagemagick, until I saw that starting with version 15, both the package and the extension were being included. Not so with 16 tho.

[kesselb]

My point is that if it is "not worth" adding the package by default, why is the pecl extension being installed?

pecl extension "imagemagick" is installed by default. svg support for imagemagick is optional and not installed by default.

[J0WI]

AFAIK imagick is used to speed up thumbnail generation and the imagemagick binary is used for extended file formats like SVG and PDF.

[pgassmann]

In nextcloud 17, imagemagick svg support is used to convert various elements like folder icons. the package is missing in the official nextcloud:17-fpm image. The folders and my logo were black after the upgrade to 17.

my workaround:

docker-compose exec nextcloud bash
apt install libmagickcore-6.q16-6-extra
./occ repair # clean frontend caches
docker-compose restart

[bkraul]

Thanks, I will try something like this in my fpm-alpine image.

[bkraul]

Actually it seems my problem is a little bit different. The SVG "seems" like it is being rendered but it is incorrectly rendered. I have checked the logs and are not able to see any helpful errors that would point to missing libs or the sort.

SVG Logo:

What it actually renders:

[zrayev]

You need to add libmagickcore-6.q16-3-extra to your image. This will add support for additional formats, including svg, to imagemagick.

For example change the part in the example where all the apt-get installs happen, to include this package:

RUN set -ex; \
    \
    apt-get update; \
    apt-get install -y --no-install-recommends \
        ffmpeg \
        smbclient \
        supervisor \
        # svg support
        libmagickcore-6.q16-3-extra \
    ; \
    rm -rf /var/lib/apt/lists/*

thanks