Skip to content

Update cryptography#219

Merged
tdviet merged 6 commits intomasterfrom
cryptography
Feb 19, 2026
Merged

Update cryptography#219
tdviet merged 6 commits intomasterfrom
cryptography

Conversation

@sebastian-luna-valero
Copy link
Collaborator

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and couldn’t find anything (or linked relevant results below)
  • If applicable, I’ve added docs and tests

Description of changes

Address https://cwe.mitre.org/data/definitions/345.html in cryptography

@sebastian-luna-valero
Copy link
Collaborator Author

I will need more time to properly fix these errors.

Initially I only wanted to update cryptography to the latest version but then bumped into other issues with pytest.

@sebastian-luna-valero sebastian-luna-valero mentioned this pull request Feb 13, 2026
5 tasks
@sebastian-luna-valero
Copy link
Collaborator Author

ok, fixed.

However, does anyone have better ideas to avoid these mocks and improve the testing?

Thanks!

@tdviet
Copy link
Owner

tdviet commented Feb 19, 2026

Thanks Sebastian

@tdviet tdviet self-requested a review February 19, 2026 14:41
@tdviet tdviet merged commit 66b945f into master Feb 19, 2026
4 checks passed
@sebastian-luna-valero sebastian-luna-valero deleted the cryptography branch February 19, 2026 14:51
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