-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add JDBC Spring Boot sample #454
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
Conversation
@Configuration | ||
public class H2Initializer { | ||
|
||
@Bean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this necessary with newer versions of Boot? I thought this was automatically setup to /h2-console
via H2ConsoleAutoConfiguration
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're absolutely correct. To make my oversight more comical I've just used Boot's H2 console auto-configuration in one of my projects a few days ago :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On a sidenote - websocket sample could also be changed to use auto-configuration for H2 console.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happens to me sometimes too. It just shows you are a long time Boot user :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On a sidenote - websocket sample could also be changed to use auto-configuration for H2 console.
Good point! I created #457
Thanks for the PR @vpavic! You are really rocking Spring Session as of late :) I provided a few comments inline. |
Thank you @rwinch :) |
c99e4b7
to
8238e91
Compare
@rwinch I've updated the PR. |
8238e91
to
85e1b8d
Compare
@rwinch I've updated the PR... again :) |
Nice work! Thanks for the PR..This is now merged into master |
I've signed the CLA.