[GitHub] [inlong] dockerzhang commented on a diff in pull request #7481: [INLONG-7480][CVE] Add a specified tomcat-embed-core dependency for fixing the CVE problem

["dockerzhang (via GitHub)" <gi...@apache.org>]

dockerzhang commented on code in PR #7481:
URL: https://github.com/apache/inlong/pull/7481#discussion_r1122518034


##########
pom.xml:
##########
@@ -215,6 +215,11 @@
                 <artifactId>flume-ng-core</artifactId>
                 <version>${flume.version}</version>
             </dependency>
+            <dependency>
+                <groupId>org.apache.tomcat.embed</groupId>
+                <artifactId>tomcat-embed-core</artifactId>
+                <version>8.5.63</version>

Review Comment:
   please add the version definition to the root `pom.xml/properties`, it will help keep the same version for other using scenes.



-- 
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@inlong.apache.org

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