Skip to content

Conversation

@mproboeuf
Copy link

Lors du moissonnage, il arrive que l'exécution de la commande wget reste bloquée.
L'ajout d'un timeout permet de limiter cet effet.

@Dolite Dolite self-assigned this Aug 20, 2025
@Dolite Dolite added the enhancement New feature or request label Aug 20, 2025
@Dolite Dolite self-requested a review September 4, 2025 11:43
@Dolite Dolite assigned mproboeuf and unassigned Dolite Sep 4, 2025
Copy link
Member

@Dolite Dolite left a comment

Choose a reason for hiding this comment

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

Comme ce réglage d'un time out est nécessaire sur d'autres commandes, on va utiliser le nom ROK4_NETWORK_TIMEOUT pour être plus générique et homogène avec les autres variables d'environnement du projet.

Il serait aussi plus clair d'utiliser l'options --timeout de wget pour exploiter cette valeur. Cela permet de garder la commande wget en principale dans l'appel.

@mproboeuf
Copy link
Author

Comme ce réglage d'un time out est nécessaire sur d'autres commandes, on va utiliser le nom ROK4_NETWORK_TIMEOUT pour être plus générique et homogène avec les autres variables d'environnement du projet.

Il serait aussi plus clair d'utiliser l'options --timeout de wget pour exploiter cette valeur. Cela permet de garder la commande wget en principale dans l'appel.

L'idée était de s'assurer que la commande wget termine proprement, ce qui n'a pas l'air d'être le cas actuellement malgré le fait que wget semble avoir un timeout (idle) par défaut de 900s (https://www.gnu.org/software/wget/manual/wget.html#Download-Options)

S'il y a un besoin plus général de s'assurer que d'autres commandes terminent proprement, je suis pour utiliser le nom plus générique de variable ROK4_SUBCOMMAND_TIMEOUT et conserver l'usage de la commande générique timeout pour s'assurer que l'exécution de la commande en paramètre terminera après le timeout qui aura été valorisé.

@mproboeuf mproboeuf removed their assignment Nov 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants