Alpine linux incompatibility

[enumag]

You recently added a FilesystemCacheFactory which uses Strings::webalize(). The problem is that webalize internally calls iconv with ASCII//TRANSLIT//IGNORE which doesn't work on Alpine.

[enumag]

To be more specific it causes a PHP notice:

PHP Notice:  iconv(): Wrong charset, conversion from `UTF-8' to `ASCII//TRANSLIT//IGNORE' is not allowed in /usr/app/vendor/nette/utils/src/Utils/Strings.php on line 185

But the worse problem is that aside of throwing this notice webalize can potentially return an empty string even if the input was not empty. Symplify doesn't actually fail, I just see the notice in the log but it is potentially dangerous.

[TomasVotruba]

Is that Symplify or Nette?

Solutions?

[enumag]

It's a problem in Nette, unfortunately there is no known solution. Even detecting the issue to throw an exception instead of bad silent behaviour is difficult.

The problem in Symplify is that it should not use a Nette function with such known issues.

As for solution I would simply use md5 instead of webalize in this case. I mean it's just a directory name for cache so the value doesn't matter, right?

[enumag]

If you're asking what to use instead of webalize to generate slugs (which is not the case here) then I can recommend cocur/slugify.

[TomasVotruba]

The namespace is easier to debug while human readble.

Could you send slugify PR for all webalize occurrences?

[enumag]

There are only two other occurrences.

1. In ProcessFactory but that seems to be a similar case as here so md5 should be fine there too.

2. In MarkdownFileDecorator. In this case you do actually need a slug, but are you sure you want to include another library for a single case? Also note that slugify works differently and in my opinion better - it works well with cyrillic, chinese and many other alphabets.

[TomasVotruba]

The namespace is easier to debug while human readble.

Same goes for paths.

I think propper solution is install the PHP extension rather than switching packages due to minor incompatible Linux version.

[enumag]

It's not a question of missing PHP extension, the extension is there but unfortunately works a bit differently than normal. It can't be solved easily.

[TomasVotruba]

Still, it's responsbility of Nette or the Linux.
It doesn't make sense to send such issue to every PHP package that uses Nette\Utils\Strings class on Alpine, does it.

Fixing it here is postponing such responsbility with band-aid instead of curing it.
Closing it here.

[enumag]

I agree but unfortunately there is no way to fix it correctly.

[TomasVotruba]

Related: nette/utils#109