lenboo commented on a change in pull request #3865:
URL:
https://github.com/apache/incubator-dolphinscheduler/pull/3865#discussion_r503022712
##########
File path:
dolphinscheduler-ui/src/js/conf/home/pages/security/pages/tenement/_source/createTenement.vue
##########
@@ -146,6 +146,13 @@
this.$message.warning(`${i18n.$t('Please enter name')}`)
return false
}
+ // Verify tenant name cannot contain special characters
+ let isSpecial = /[~#^$@%&!*()<>《》:;'"{}【】 ]/gi
+ if (isSpecial.test(this.tenantName)) {
+ this.$message.warning(`${i18n.$t('Please enter tenant name without
special characters')}`)
+ return false
+ }
Review comment:
i think we should keep the front and back consistent, the api module
should also limit the tenant name.
on the other hand, should we limit tenant name just 'number and character'?
----------------------------------------------------------------
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:
[email protected]
