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

fix encryption errors #295

Closed
wants to merge 1 commit into from
Closed

Conversation

ajuniper
Copy link

@ajuniper ajuniper commented Nov 3, 2019

These fixes address the issues seen in issue #294
I'm not a Python expert so they might be solid in all environments but they work for me...

@charliejllewellyn
Copy link

Nice thanks for this fix, it worked for me :) There is an error on line 170 though where you have an extra "("

return encryptor.update((bytes(payload)) + encryptor.finalize() should be
return encryptor.update(bytes(payload)) + encryptor.finalize()

Hopefully someone merges this soon :)

@felipediel
Copy link
Collaborator

HI @ajuniper. It seems that this problem still exists. Could you please resolve the conflicts so we can merge?

@felipediel felipediel mentioned this pull request Jan 11, 2021
@felipediel
Copy link
Collaborator

I resolved conflicts and merged. Thank you very much @ajuniper!

@felipediel felipediel closed this Jan 11, 2021
@ajuniper
Copy link
Author

@felipediel Apologies for not commenting sooner, been AFK. Glad you resolved the conflicts.

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.

3 participants