Bump spring-boot from 3.5.5 to 4.0.4 [#2625]#2588
Bump spring-boot from 3.5.5 to 4.0.4 [#2625]#2588JoranVanBelle wants to merge 1 commit intocomixed:mainfrom
Conversation
600b6aa to
0fc47ba
Compare
0fc47ba to
215f7cf
Compare
215f7cf to
1276ba7
Compare
f6d299f to
a5b5d98
Compare
mcpierce
left a comment
There was a problem hiding this comment.
@JoranVanBelle Looks very good, thank you!
Please address the requested changes, and also the issues reported by the automated tests so that they all pass. I saw four issues reported by SonarQube, though two of them are filename issues that seemed off.
Once those issues are addressed, I think this is ready to merge.
a83c1ab to
4522215
Compare
|
@mcpierce I resolved all the remarks (if I didn't miss one). Except the one of sonarQube, he complains about tests being similar. What should I do about those? |
I went ahead and marked that as Acceptable. |
2c9159b to
4664c9f
Compare
4664c9f to
1809998
Compare
|
@JoranVanBelle Are you able to run the build? I just tried building and running it locally and it's failing with the following: |
1809998 to
f163f77
Compare
|
@JoranVanBelle Are you able to run things locally? I'm generating the JAR files and attempting to run it and am hitting this issue now: |
908a707 to
62a4d6e
Compare
97e5033 to
760cb7a
Compare
|
@JoranVanBelle Please do a build and run the jar file locally to see if you're hitting the same issue I am. When I run with the cmdline:
I'm getting this output and the app fails to start: |
|
@mcpierce My bad, I saw the tests were green and I assumed everything was fine. |
f992421 to
914ded1
Compare
|
@JoranVanBelle Definitely getting closer! Are you able to login? When I try, I get the following: (eta) Separating out the frontend from the build, I'm seeing this when Angular tries to proxy to the upgraded server: It seems the upgrade is disallowing connections. |
914ded1 to
6e14114
Compare
|
@mcpierce think it's fixed. When going to |
There was a problem hiding this comment.
@JoranVanBelle What end-to-end testing have you been running for this? For my tests I:
- verify the PR:
mvn clean verify - build a runnable jar file:
mvn clean package - run the jarfile:
java -jar comixed-app/target/comixed-app-3.3-SNAPSHOT.jar --logging.level.root=debug --spring.h2.console.enabled=true - interact with the system:
- check the schema in the dbconsole
- import comics
- scrape comics
- run batch jobs (organize, scrape, recreate)
- create/edit/delete reading lists
Would you mind running a few of those system interactions on your end to see if they work for you. When I try to access the dbconsole using H2 the system is redirecting me to the login page (it shouldn't do that in local mode). I'm still testing the others, and things are generally looking good at this point.
Also, could you please add to the UPGRADING.md file a note about going from v3 -> v4 that users will have to drop the Spring batch tables? We had similar notes for 1.6.1 -> v2.0.
8d8f5eb to
4dead9c
Compare
4dead9c to
ebd2bf9
Compare
|
|
@mcpierce could you give it another look? The h2-console is back and I had some interactions with the system as well. I also copied the message from the previous migration and changed the version numbers to the most recent versions |





Fixes #2625
Status
READY
Does this PR contain migrations?
YES
Before You Submit Your PR: