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.

Sign up for GitHub

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

Jump to bottom

Also raise ImproperlyConfigured for Provider.symkey bad key length #367

Closed

decentral1se opened this issue Jun 5, 2017 · 0 comments

Labels

Milestone

Contributor

decentral1se commented Jun 5, 2017

Following from #324 (comment):

Currently, you can set a symkey like 'ab' and all the setup/init works, but you blow up at runtime, when you get FailedAuthentication or other AssertionErrors from aes.encrypt(). So it would be user friendlier to prevent a too short to be useful symkey too, instead of just empty and None.

decentral1se added the enhancement label

decentral1se added this to the P1: MUST milestone

decentral1se mentioned this issue

Make symkey an optional kwarg. #324

Merged

2 tasks

decentral1se self-assigned this

tpazderka unassigned decentral1se

decentral1se closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment