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

Add a check on the length of replies and outbound SMSes. #19

Open
hodgestar opened this issue Jun 11, 2013 · 0 comments
Open

Add a check on the length of replies and outbound SMSes. #19

hodgestar opened this issue Jun 11, 2013 · 0 comments

Comments

@hodgestar
Copy link
Contributor

The check would be on a configurable maximum message length (set in the config for the JS app) and would log a warning if the content is too long.

The test harness already checks this but a check in production would help pick up odd errors (e.g. where the aggregator ends the USSD session because the reply is too long) more quickly. Sometimes production replies are longer than test replies (e.g. because dynamic content is included in the replies).

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