tlocke / scramp

Python implementation of the SCRAM protocol
MIT No Attribution
28 stars 5 forks source link

Raise ScramException for malformed messages #11

Closed half-duplex closed 2 years ago

half-duplex commented 2 years ago

ProgVal pointed out that scramp doesn't catch KeyErrors for some malformed messages, which makes ensuring reliability and polite failures more challenging. This attempts to catch errors like that and raise a descriptive ScramException instead.

tlocke commented 2 years ago

Thanks for this patch @half-duplex, I've included it in the latest release 1.4.4. I'll close this PR, but let me know if there are still problems.