You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by GitBox <gi...@apache.org> on 2020/11/24 15:54:57 UTC
[GitHub] [ambari] hapylestat commented on a change in pull request #3259: [AMBARI-25583] The notes field in the config history page does not support UTF-8 characters for filtering
hapylestat commented on a change in pull request #3259:
URL: https://github.com/apache/ambari/pull/3259#discussion_r529672627
##########
File path: ambari-web/app/utils/validator.js
##########
@@ -264,7 +264,7 @@ module.exports = {
return true;
};
if (/^[\?\|\*\!,]/.test(value)) return false;
- return /^((\.\*?)?([\w\s\[\]\/\?\-_,\|\*\!\{\}\(\)]*)?)+(\.\*?)?$/g.test(value) && (checkPair(['[',']'])) && (checkPair(['{','}']));
+ return /^((\.\*?)?([\w\s\[\]\/\?\-_,\|\*\!\{\}\(\)]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF]]*)?)+(\.\*?)?$/g.test(value) && (checkPair(['[',']'])) && (checkPair(['{','}']));
Review comment:
The diapasons here are related to:
- Latin block (characters from dead language and used in medicine, non English characters)
- CJK with Latin (wrong and not fully covered)
- variation of symbols (non characters) and some Arabic range
Please referrer here for correct CJK ranges:
https://stackoverflow.com/questions/1366068/whats-the-complete-range-for-chinese-characters-in-unicode
and you might include Japanese ranges as well
To verify what is in the range, could be used (found with google) `https://qaz.wtf/u/show.cgi`. Mode `Hexadecimal range`, value for example : `\u00A0-\uD7FF` -> `00A0-D7FF`
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org