Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignorer les fichiers pour lesquels on sait que ça ne va pas marcher #17

Open
DavidBruant opened this issue Jan 3, 2018 · 2 comments
Open

Comments

@DavidBruant
Copy link
Member

DavidBruant commented Jan 3, 2018

Dans la version en ligne de commande, on traite tous les fichiers du inDir

readdir(inDir)
.then(files => {
console.log(chalk.underline('Files found'), files);
const limit = pLimit(10);
return Promise.all( files.map(f => limit(() => anonymizeOneFile(f))) );
})

On ne vérifie pas qu'il s'agit d'un fichier xml ou avec le bon encoding, ou avec le bon format

Et donc, ça peut faire perdre un peu de temps de traiter ces fichiers s'ils sont gros (mais ça ne perturbe pas le traitement des fichiers corrects)

@DavidBruant
Copy link
Member Author

Traiter le cas des dossiers (voir #21 )

@DavidBruant
Copy link
Member Author

Un deuxième cas problématique potentiel est si le fichier est un fichier XML mais qui n'a rien à voir. Le niveau de validation ultime serait la validation contre les XML Schema

@DavidBruant DavidBruant changed the title Voir ce qu'il se passe quand il y a des fichiers pas XML dans le dossier Ignorer les fichiers pour lesquels on sait que ça ne va pas marcher Jan 4, 2018
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

No branches or pull requests

1 participant