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

638-rebased #1

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

638-rebased #1

wants to merge 62 commits into from

Conversation

arp102
Copy link
Owner

@arp102 arp102 commented Aug 29, 2023

No description provided.

tirkarthi and others added 30 commits March 16, 2022 14:14
This is often required for hostname verification.
Pass the required argument to the `build_cli_parser` function where
it was missed. Pass the missed argument in the `objects.SecretData`
initialization.
This should provide a useful project description on PyPI the next time
we publish a release.
…ded sudo in run.sh, updated server.conf to remove db, updated tests and comments to reflect app specific info being a list
The cryptography release 39.0.0 added a new parameter to the
backend.load_pem_private_key and backend.load_der_private_key
that's required. This patch uses the serialization method to load keys
because there the new parameter is optional.

https://cryptography.io/en/latest/changelog/#v39-0-0

This patch fixes the tests test_encrypt_decrypt_asymmetric
This test is meant to intentionally trigger an exception in the cryptography library
by creating a CMAC with a non-block cipher algorithm, IDEA.
That doesn't work any more because IDEA is now treated as a block cipher algorithm.
To fix this, we now use the ARC4 algorithm instead,
which does trigger the expected exception.
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.