slawekjaranowski commented on code in PR #89: URL: https://github.com/apache/maven-dependency-analyzer/pull/89#discussion_r1233373650
########## src/main/java/org/apache/maven/shared/dependency/analyzer/asm/DependencyClassFileVisitor.java: ########## @@ -75,6 +75,9 @@ public void visitClass(String className, InputStream in) { // some bug inside ASM causes an IOB exception. Log it and move on? // this happens when the class isn't valid. logger.warn("Unable to process: " + className); + } catch (IllegalArgumentException e) { + // [MSHARED-1248] should log instead of failing when analyzing a corrupted jar file + logger.warn("Unable to process: " + className, e); Review Comment: I'm still not sure if it will be the best solution. Eg. when analyzer will not support jdk in newer version - user will only have a warnings but nothing will be analyzed. We know that warnings are many times ignored ... -- 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: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org