etalab / transport-tools

GNU Affero General Public License v3.0
4 stars 2 forks source link

Ajout du convertisseur GTFS vers NeTEx (beta) #9

Closed thbar closed 2 years ago

thbar commented 2 years ago

Ça n'a pas été simple, du fait de la présence de la librairie proj qui introduit une vraie complexité de compilation.

Je m'appuie pour l'instant sur une image rust-ci spécifique fournie par Kisio Digital (merci !), dont une variante dispose de proj. Voir:

On pourrait plus tard alléger le dispositif en allant chercher uniquement les "artefacts" pour rust via https://hub.docker.com/r/kisiodigital/proj-ci, mais ça semble faire le travail.

Point restants

Note qualité

À considérer comme une beta : il y a des warnings sur "proj.db", je vais shipper en les ignorant, mais on verra si ça pose problème durant les tests (voir https://github.com/etalab/transport-tools/issues/10).

Note sécurité

Du coup on est dépendant d'un point de vue sécurité de ce qui est fait sur la publication des images là-bas, au moins pour l'instant.

Je pense préférable à terme de recompiler notre propre image pour réduire un peu la surface d'attaque, car ce n'est pas très compliqué, et peu fréquent.

Voir ticket https://github.com/etalab/transport_deploy/issues/47

thbar commented 2 years ago

@fchabouis tu peux jeter un oeil rapidement ? Si je merge ça ça permettra de publier l'image, puis que j'aille l'intégrer de l'autre côté dans transport-ops puis dans transport-site pour ne pas te retarder.

thbar commented 2 years ago

Elle a l'air chouette cette librairie

C'est riche 😄 ; merci pour la review ! Je vais avancer sur le lego suivant.