Skip to content

FIX default configuration with a context path akin to the one used in production.#17

Merged
edegenetais-nx merged 1 commit intomainfrom
bug/16_bad_default_configuration
Mar 21, 2024
Merged

FIX default configuration with a context path akin to the one used in production.#17
edegenetais-nx merged 1 commit intomainfrom
bug/16_bad_default_configuration

Conversation

@edegenetais-nx
Copy link
Copy Markdown
Contributor

Closes #16

@edegenetais-nx edegenetais-nx requested a review from bou3108 March 20, 2024 18:30
@edegenetais-nx edegenetais-nx self-assigned this Mar 20, 2024
@edegenetais-nx
Copy link
Copy Markdown
Contributor Author

@bou3108 : a priori cette valeur par défaut est conforme à ce qui sert en production, si elle est utilisée par défaut ça permet de faire tourner les services en éclaté sur un poste.

Merge par défaut : vendredi 22 mars 2024

Comment on lines 16 to 20

server.servlet.context-path=v1
server.servlet.context-path=/toggle/v1
api.base.url=http://localhost:9000/api

spring.servlet.multipart.max-file-size=10MB
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En déployant via Nomad, on surcharge systématiquement l'application.properties via un template embarqué dans le descripteur Nomad.
Effectivement pour tout faire tourner localement il faut le modifier ici, mais alors je ne suis pas sûr qu'on ait bien défini des ports différents pour chaque composant. Notamment je ne suis pas sûr que le port par défaut de l'api soit 9000, on est sans doute sur 8080 (comme toggle)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En déployant via Nomad, on surcharge systématiquement l'application.properties via un template embarqué dans le descripteur Nomad. Effectivement pour tout faire tourner localement il faut le modifier ici, mais alors je ne suis pas sûr qu'on ait bien défini des ports différents pour chaque composant. Notamment je ne suis pas sûr que le port par défaut de l'api soit 9000, on est sans doute sur 8080 (comme toggle)

OK, je verrai quand j’ajouterai l'API aux backends appelés par le front (actuellement le seul appel est l'endpoint check pour prouver que le proxying marche)

@edegenetais-nx edegenetais-nx merged commit 15f90f3 into main Mar 21, 2024
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.

Bad internal application.properties values prevent service startup on default configuration.

2 participants