Sprint 42 upgrade weaver webservice 2.1.1 rc7#43
Sprint 42 upgrade weaver webservice 2.1.1 rc7#43rmathew1011 merged 22 commits intosprint-upgrade-stagingfrom
Conversation
src/main/java/edu/tamu/app/controller/ButtonManagementController.java
Outdated
Show resolved
Hide resolved
src/main/java/edu/tamu/app/controller/ButtonManagementController.java
Outdated
Show resolved
Hide resolved
src/main/java/edu/tamu/app/controller/ButtonManagementController.java
Outdated
Show resolved
Hide resolved
…b.com:TAMULib/GIFMButtonService into sprint_42_upgrade_weaver_webservice_2.1.1-rc7
kaladay
left a comment
There was a problem hiding this comment.
The tests pass.
However, I have just attempted to run this using mvn clean spring-boot:run with Java 11 and this fails to start up.
08:30:17.205 [main] ERROR org.springframework.boot.SpringApplication - Application run failed
java.lang.IllegalArgumentException: Could not resolve placeholder 'catalina.home' in value "${catalina.home}/logs/application.log"
I'm wondering if this is the result of changing it from a war to a jar.
Does catalina.home now need to be added to the config?
The log path needs to change. Catlina home is only applicable to tomcat deployment. |
|
I suggest the following changes for making the tests easier to read when running. Create the file |
Resolves #42