Setup spring boot #2
Conversation
culka
force-pushed
the
setup-spring-boot
branch
4 times, most recently
from
0e14b5a to
05c41d8
Compare
Use runner with Java 24 Create mandatory config file
culka
force-pushed
the
setup-spring-boot
branch
12 times, most recently
from
dcf2a33 to
c737675
Compare
Huulivoide
left a comment
Huulivoide
left a comment
There was a problem hiding this comment.
Pintapuolin kyllä vaikuttaa ihan OK:lta, noita paria ehdotusta lukuun ottamatta.
Dockerfile
Outdated
| EXPOSE 8080 | ||
|
|
||
| # download script for reading Docker secrets | ||
| RUN curl -o /tmp/read-secrets.sh "https://raw.githubusercontent.com/HSLdevcom/jore4-tools/main/docker/read-secrets.sh" |
There was a problem hiding this comment.
ADD https://raw.githubusercontent.com/HSLdevcom/jore4-tools/main/docker/read-secrets.sh /tmp/read-secrets.shNiin voi pudottaa curlin asennuksen kokonaan pois. Vois tiketöidä ja tehdä saman muutoksen muihinkin docker filuihin.
| # Datasource settings of the Jore 4 database | ||
| jore4.db.driver=org.postgresql.Driver | ||
| jore4.db.url=jdbc:postgresql://localhost:6432/jore4e2e?stringtype=unspecified | ||
| jore4.db.username=dbhasura |
There was a problem hiding this comment.
Hasura käyttäjällä on myös kirjotus oikeidet. Tän palvelun pitäs varmaan lähtökohtasesti vaan lukee kamaa kannasta, niin vois olla joku oma käyttäjä, jolla vaan sopivat lukuoikeudet.
There was a problem hiding this comment.
Ihan totta. CI profiilia käytetään vain testiajoissa ja kantaan ei ole luotu vielä oikeaa käyttäjää netex exporterille, joten tämä vaihdetaan kun käyttäjä on. Nykyisiä alustuksia ei kannata lähteä muokkaamaan vaan tehdä uusi käyttäjä vasta yhdistettyyn kantaan.
culka
force-pushed
the
setup-spring-boot
branch
from
c737675 to
4272d2f
Compare
Update dependencies and add all needed by Spring Boot, jOOQ and Jackson.
Update Java to 24.
Add configuration for database connection.
Generation for jOOQ classes.
Add KtLint code checking.