View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0023214||mantisbt||performance||public||2017-08-13 05:18||2017-08-13 14:38|
|Priority||normal||Severity||minor||Reproducibility||have not tried|
|Target Version||Fixed in Version|
|Summary||0023214: Remove usage of outdated phputf8 library|
Advantages of removing the library and enforcing mbstring extension
|Tags||No tags attached.|
Not sure we can target this to 2.x or have to wait for 3.x.
We use mantis on a server without support for mbstring, and one that we can not add mbstring to.
You may find that the library hasn't changed as there is no need - if the library correctly counts the length of a utf8 string the code is not going to change!!
It's not implemented at the moment, but it's documented that mbstring is mandatory https://mantisbt.org/docs/master/en-US/Admin_Guide/html-desktop/#admin.install.requirements.software
I looked at the code and found that even the current implementation needs the extension, at least if you want to use Markdown.
@fudge, why can't you add the extension?
There's a couple of internal reasons (Both technical and non-technical), externally the extension has had a few security issues recently (for example 3 CVE's in June).
In terms of the parsdown library, that library currently has several utf8 handling issues, somewhat surprised you've not hit issues, but then it may be that english is the predominate language and/or that the utf8 library also processing the text is hiding them
I see no way and no reason to consider internal reasons as long as they are internal.
I don't see at the moment that there is a special reason not to use mbstring.
|2017-08-13 05:18||atrol||New Issue|
|2017-08-13 05:18||atrol||Status||new => assigned|
|2017-08-13 05:18||atrol||Assigned To||=> atrol|
|2017-08-13 05:18||atrol||Description Updated||View Revisions|
|2017-08-13 05:20||atrol||Note Added: 0057447|
|2017-08-13 08:09||fuge||Note Added: 0057448|
|2017-08-13 08:58||atrol||Relationship added||related to 0021959|
|2017-08-13 09:17||atrol||Note Added: 0057449|
|2017-08-13 09:18||atrol||Note Edited: 0057449||View Revisions|
|2017-08-13 10:57||fuge||Note Added: 0057451|
|2017-08-13 14:38||atrol||Note Added: 0057452|