Skip to content

Commit

Permalink
[Fix] Fix emoji joiner FP
Browse files Browse the repository at this point in the history
Issue: #4290
  • Loading branch information
vstakhov committed Oct 3, 2022
1 parent 572e0b0 commit e5328bd
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/libutil/cxx/utf8_util.cxx
Expand Up @@ -85,8 +85,10 @@ rspamd_normalise_unicode_inplace(char *start, size_t *len)
if (!zw_spaces.isFrozen()) {
/* Add zw spaces to the set */
zw_spaces.add(0x200B);
/* TODO: ZW non joiner, it might be used for ligatures, so it should possibly be excluded as well */
zw_spaces.add(0x200C);
zw_spaces.add(0x200D);
/* See github issue #4290 for explanation. It seems that the ZWJ has many legit use cases */
//zw_spaces.add(0x200D);
zw_spaces.add(0xFEF);
zw_spaces.add(0x00AD);
zw_spaces.freeze();
Expand Down

0 comments on commit e5328bd

Please sign in to comment.