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

Ne pas lancer des erreurs pour negotiateType #519

Open
srosset81 opened this issue Jan 18, 2021 · 2 comments
Open

Ne pas lancer des erreurs pour negotiateType #519

srosset81 opened this issue Jan 18, 2021 · 2 comments

Comments

@srosset81
Copy link
Contributor

srosset81 commented Jan 18, 2021

Problématique
Il faudrait éviter que la fonction negotiateType ne lance une erreur pour qu'on ait pas à mettre des try...catch partout.

Composants concernés
Fonction negotiateType dans le package @semapps/mime-types.
Il faut aussi adapter toutes les fonction negociate du package qui se basent sur cette même fonction.

Proposition
Elle pourrait simplement retourner false quand aucun type n'est trouvé ?

@simonLouvet
Copy link
Contributor

je ne sais pas pourquoi cela te dérange de gérer du try catch. pas d'objection à cette modification.

@srosset81
Copy link
Contributor Author

Parce que ça a occasionné un bug récemment, et puis parce que ça alourdit le code en général.

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

2 participants