[GitHub] [nifi] mcgilman commented on a diff in pull request #6800: NIFI-10997 - Ensure auditing of process group / controller service op…

[GitBox <gi...@apache.org>]

mcgilman commented on code in PR #6800:
URL: https://github.com/apache/nifi/pull/6800#discussion_r1054513933


##########
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/pom.xml:
##########
@@ -346,6 +346,12 @@
             <artifactId>jersey-spring5</artifactId>
             <scope>provided</scope>
         </dependency>
+        <dependency>
+            <groupId>org.springframework.boot</groupId>
+            <artifactId>spring-boot-test</artifactId>
+            <version>2.7.4</version>

Review Comment:
   Is there a specific reason 2.7.4 was chosen? It appears that 2.7.4 depends on `spring-core` and `spring-context` 5.3.23 [1]. However, NiFi is already on `spring.version` 5.3.24. I think we bump to 2.7.6 which depends on 5.3.24.
   
   https://repo1.maven.org/maven2/org/springframework/boot/spring-boot/2.7.4/spring-boot-2.7.4.pom



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

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