[ https://issues.apache.org/jira/browse/MSHARED-1297?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17755436#comment-17755436 ]
ASF GitHub Bot commented on MSHARED-1297: ----------------------------------------- slawekjaranowski commented on code in PR #42: URL: https://github.com/apache/maven-archiver/pull/42#discussion_r1296895957 ########## src/main/java/org/apache/maven/archiver/MavenArchiver.java: ########## @@ -800,6 +800,21 @@ public static Optional<Instant> parseBuildOutputTimestamp(String outputTimestamp } } + private static boolean isNumeric(String str) { + + if (str.isEmpty()) { + return false; + } + + for (char c : str.toCharArray()) { + if (!Character.isDigit(c)) { Review Comment: It is simple copy method from plexu-utils ... https://github.com/codehaus-plexus/plexus-utils/blob/master/src/main/java/org/codehaus/plexus/util/StringUtils.java#L1749 I think it is enough for our case - if user provide a special chars Long.parseLong simply throw exception if needed. > Prefer JDK features over plexus-utils, plexus-io > ------------------------------------------------ > > Key: MSHARED-1297 > URL: https://issues.apache.org/jira/browse/MSHARED-1297 > Project: Maven Shared Components > Issue Type: Task > Components: maven-archiver > Reporter: Slawomir Jaranowski > Assignee: Slawomir Jaranowski > Priority: Major > Fix For: maven-archiver-3.6.1 > > -- This message was sent by Atlassian Jira (v8.20.10#820010)