search

EsupPortail / Esup-Pod

plateforme de gestion de fichier vidéo
https://pod.esup-portail.org/
GNU Lesser General Public License v3.0
36 stars 77 forks source link

[DONE] run development server on port 8000 #1125

Closed azmeuk closed 5 months ago

azmeuk commented 5 months ago

For the sake of simplicity I suggest to run the development server on port 9090 instead of 8080 inside docker containers, to align this with the exposed port. Unless this is motivated by technical reasons I ignore?

I found by working on #1058 that this is sometimes confusing to deal with different ports depending on the context. Communications among docker containers had to use the port 8080 (for instance communication between a peertube and a pod instance) whether pod had to be externally reached with the 9090 port (with my browser).

The initial data for Site use the 9090 port. https://github.com/EsupPortail/Esup-Pod/blob/be947445e4bd649d6e199c9a9cbf218d79a7d498/pod/main/fixtures/initial_data.json#L2-L8 For the AP implementation we rely on Site to build absolute URLs. That would lead to funky situations with the peertube federation for instance, when a pod AP endpoint accessed by the 8080 port would expose a 9090 URL.

I put this changes on a separate PR than the incoming one for the AP implementation to make it easier to review. If you accept this PR I intend to submit another one that would uniformise the docker host names in the same way.

What do you think?

ptitloup commented 5 months ago

Bonjour, cela ne semble pas poser de problème. Je vois avec la personne qui a mis en place les docker s'il y avait une raison particulière pour du mapping 9090:8080 et je reviens vers vous rapidement (d'ici cette fin de semaine)

ptitloup commented 5 months ago

Bonjour, après avoir fait le tour des contributeurs, pas de soucis pour nous pour aligner les ports. Le port par défaut de django est le 8000. On pourrait aligner sur ce dernier si cela vous va ? Sinon ok pour mettre 9090. Bonne journée

azmeuk commented 5 months ago

On pourrait aligner sur ce dernier si cela vous va ?

Oui pas de souci. Je vais mettre à jour la PR pour utiliser le port 8000 dans ce cas.

ptitloup commented 5 months ago

On pourrait aligner sur ce dernier si cela vous va ?

Oui pas de souci. Je vais mettre à jour la PR pour utiliser le port 8000 dans ce cas.

Merci !

azmeuk commented 5 months ago

Et voilà !