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

Password max length 15 #24

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

SethaGitHub
Copy link

SMPP password max length should be 15

DomAmato and others added 4 commits April 27, 2020 14:44
* Fix bug in length value not resetting after decoding

also added a type for network error codes to better show what they are, even though the spec is not honored in most implementations.

* remove old python versions

* add a test to do decode and encoding with the same encoder

* add test to check if patch works

uncommenting the code fix causes the test to fail

* make field names of named tuples easier to read
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

Successfully merging this pull request may close these issues.

2 participants