You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2022/04/23 09:30:50 UTC

[GitHub] [dolphinscheduler] zhongjiajie commented on a diff in pull request #9634: [Fix-9633][TenantManage/CreateTenant] Create a tenant is prompting an…

zhongjiajie commented on code in PR #9634:
URL: https://github.com/apache/dolphinscheduler/pull/9634#discussion_r856872528


##########
dolphinscheduler-api/src/test/java/org/apache/dolphinscheduler/api/utils/RegexUtilsTest.java:
##########
@@ -28,10 +28,10 @@ public class RegexUtilsTest {
     @Test
     public void testIsValidLinuxUserName() {
         String name1 = "10000";
-        Assert.assertFalse(RegexUtils.isValidLinuxUserName(name1));
+        Assert.assertTrue(RegexUtils.isValidLinuxUserName(name1));

Review Comment:
   I find out that you change de previous logic, and it support the user name start with number now?



-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org