[
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)
