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

Externalize config #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

danlangford
Copy link

@danlangford danlangford commented Mar 22, 2018

to address #2 this updates the codebase to a more current version of Spring Boot. i also added some best practices to the pom file and spring project configuration. most importantly i migrated from Test.properties to application.properties in an attempt to quickly take advantage of Springs built in support for externalized configuration. all you have to do is create your own application.properties in the same directory as the jar and Spring will consider it when looking up properties, and consider it in a high priority than the files in the jar. https://docs.spring.io/spring-boot/docs/current/reference/html/boot-features-external-config.html. Also another major change was the use of Spring @ConfigurationProperties to quickly map all the properties to a type strict POJO for easy access.

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