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

🛡️ Sortir en erreur sans appeler l'API si le sms fait plus de 150 caractères #21

Open
adriens opened this issue Jun 24, 2024 · 0 comments
Assignees
Labels
BUILD Toute activité liée à la création de nouvelles fonctionnalités enhancement New feature or request

Comments

@adriens
Copy link
Member

adriens commented Jun 24, 2024

❔ A propos

Actuellement le cli ne teste pas la longueur du message, ce qui :

  • Fait tout de même un appel (pour rien)
  • Sort en erreur 413

🎯 Action

Implémenter un test sur la longueur du message < 150 et sortir en erreur sinon... avec une log explicite

💰 Bénéfices

  • L'utllisateur final ne gâche pas de token inutilement et peut produire plus de valeur avec la même somme
  • On évite le traffic inutile sur l'API
@adriens adriens self-assigned this Jun 24, 2024
@adriens adriens added BUILD Toute activité liée à la création de nouvelles fonctionnalités enhancement New feature or request labels Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUILD Toute activité liée à la création de nouvelles fonctionnalités enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant