Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upgraded Spring Boot from 2.3.0_Release to 3.0.1 and Java from 11 to 17 #25

Open
wants to merge 19 commits into
base: db-test-context
Choose a base branch
from

Conversation

PierreSQS
Copy link

Dear John,

I've upgraded the code about your nice course on Spring Security (please follow the link).

Following the changes in case you're interested in:

  • upgraded SB to 3.0.1

  • upgraded Java to 17

  • upgraded the MapStruct to 1.5.3_Final

  • upgraded Lombok to the official SB3.0.1 Version

  • adapted the POM accordingly

  • the Security Config has drastically changed since as per SB3.0.0 the WebSecurityConfigurerAdapter has been deprecated an they encourage a Bean Model in the Security Filter Chain!

  • the Hibernate Packages have been migrated to jakarta.persistence.*

  • In the H2 embedded DB, User is a reserved word (as per H2.2.214). I've added an @Table - annotation on the User-Entity to make it work. (See the User Entity).

  • The Hibernate @Type-Annotation has changed. Therefore I've removed the @Type(type="org.hibernate.type.UUIDCharType") annotation since doesn't work and no more needed

  • As per SB 2.4.2 and Thymeleaf 3.0.12 static call in the Thymeleaf-Templates has been restricted (Please check the createBeer.html and createOrUpdateBeer.html templates)

  • I didn't updated the Webjars dependencies and Co.

I've done some optimizations in the code:

  • cleaned code in general so far possible
  • I've optimized the @SpringBootTest-Annotation and Test settings
  • I've optimized some deprecated Thymeleaf-fragments in the Layout Templates (Please see the fragments/layout.html)
  • allowing the h2-console path doesn't work anymore with .antMatchers("/h2-console/**").permitAll() I've replaced it with .requestMatchers(PathRequest.toH2Console()).permitAll()

You can see the changes in more details on this branch in my Repo. I've added some comments with Pics on some commits.
I hope you could appreciate it.

Many thanks for your so cool courses

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant