Produits Entr'ouvert » w.c.s.

Pour info côté démarches simplifiées, le comportement a récemment été modifié, https://github.com/demarches-simplifiees/demarches-simplifiees.fr/issues/9733 :

Pour l'usager il y avait précédemment un retour pour lui indiquer que le fichier n'était pas bon, ça a été retiré (de toute façon le fichier est déjà chez l'usager, tant pis pour lui).

Pour l'agent le nom du fichier apparait mais sans lien pour le télécharger tant que l'antivirus n'est pas passé sur le fichier. (et s'il est passé et que le fichier est mauvais, ça reste pas téléchargeable + une note d'explication).

On pourrait imaginer en plus, pour les admins, pour analyse, avoir un lien de téléchargement qui fournisse le fichier dans un zip chiffré.

Je me dis qu'on pourrait faire encore plus simple, uniquement de l'affichage (antivirus pas passé/anti-virus passé et OK/antivirus passé et NOK faites attention) sans restriction sur le téléchargement.

La restriction du téléchargement entraîne des complications de traitement de la demande, je me dis qu'on serait mieux à ne pas mettre un doigt là dedans.

@Pierre je veux bien reprendre ce ticket.

La restriction du téléchargement entraîne des complications de traitement de la demande, je me dis qu'on serait mieux à ne pas mettre un doigt là dedans.

Je pense important une fois qu'un fichier est connu comme vérolé de ne pas pouvoir le télécharger, je ne vois pas ce que ça change au traitement de la demande ? (entre "ne pas pouvoir le télécharger" et "compter sur l'agent qui va voir que c'est écrit 'ne pas cliquer' à côté du fichier").

Frédéric Péters a écrit :

Je pense important une fois qu'un fichier est connu comme vérolé de ne pas pouvoir le télécharger, je ne vois pas ce que ça change au traitement de la demande ? (entre "ne pas pouvoir le télécharger" et "compter sur l'agent qui va voir que c'est écrit 'ne pas cliquer' à côté du fichier").

Mes 2 cents sur l'affaire. C'est moi qui a dit que pour que le traitement des demandes des citoyens reste équitable, il faudra éviter que des demandes ne puissent pas être traitées à cause de l'anti-virus. Mais on peut aussi imaginer un mode "trouille" où on désactive effectivement le chargement des fichiers non-scannés, ou détectés comme virus, ou pas scannés depuis X heures.

• document jamais scanné : « ce document n'est pas encore passé dans l'anti-virus, vous ne pouvez pas le télécharger »
• scanné récemment : « ce document a été scanné le x/x/x à xhx par l'anti-virus » (et le lien vers le download)
• scanné il y a plus de X heures (configurable par tenant) : « ce document a été scanné le x/x/x à xhx par l'anti-virus, un nouveau scan va être programmé », pas de lien ... (+ un lien pour lancer un scan ?)
• (et bien sûr le cas du fichier scanné et détecté vérolé, là c'est plus facile)

A voir ensuite avec quelle fréquence on programme le scan des documents, de fait un truc qui tourne en permanence serait idéal, avec une parallélisation sur "n" tenants en simultané, toute optimisation bienvenue... à voir éventuellement ce que clamav/clamd sait faire sur un "pool" de fichiers, si ça optimise ou pas...

Tout cela en plus, bien sûr, d'un scan au moment de la réception du fichier, en asynchrone (worker) ?

Emmanuel Cazenave (ecazenave) a ouvert une pull request sur Gitea concernant cette demande :

• URL : https://git.entrouvert.org/entrouvert/wcs/pulls/1415
• Titre : WIP: misc: scan uploads with clamd (#87739)
• Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/1415/files

Emmanuel Cazenave (ecazenave) a fermé une pull request sur Gitea concernant cette demande.

Je commence le travail sur l'interface et voici ce que je m'apprête à faire.

Toucher aux deux seules vues de backoffice qui permettent de télécharger des fichiers (sauf erreur de ma part) :

• /backoffice/management/monformulaire/XX/download?f=ZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZ
• /backoffice/management/monformulaire/XX/attachment?f=YYYYYYYYYYYYYYYYYYYYYYYYYYYYYY

Si un fichier est scanné et pas de virus, RAS ça fonctionne comme d'habitude.

Si pas scanné ou scanné et vérolé on redirige vers une page qui:

• explique la situation
• permet de revenir d'où l'on vient (via le referer de la requête d'origine)
• permet de télécharger quand même si on a je ne sais quelle permission

Je pense qu'il faut dès le début apporter les mêmes restrictions en front, surtout qu'a priori c'est le même code et donc limite plus facile de faire ainsi.

permet de revenir d'où l'on vient (via le referer de la requête d'origine)

Oublie le referer, c'est une mauvaise donnée, simplement javascript:back() s'il faut vraiment un lien de retour.