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.

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

client: Don't panic if a cryptography operation fails #172

Merged
merged 1 commit into from
Dec 9, 2024
Merged

client: Don't panic if a cryptography operation fails #172

merged 1 commit into from
Dec 9, 2024

Conversation

bilelmoussaoui
Copy link
Owner

See #171

@bilelmoussaoui bilelmoussaoui mentioned this pull request Dec 9, 2024
Open
@bilelmoussaoui bilelmoussaoui merged commit f198bcc into main Dec 9, 2024
7 checks passed
@bilelmoussaoui bilelmoussaoui deleted the bilelmoussaoui/crypto-fail branch December 9, 2024 13:59
@cole-miller
Copy link

Thanks for this @bilelmoussaoui, really appreciate it.

@cole-miller
Copy link

If it's not too much trouble it'd be great to have this in a release!

@bilelmoussaoui
Copy link
Owner Author

I am waiting for the server side implementation to progress a bit more to avoid introducing more API breaks. So I would recommend using oo7 from git for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bilelmoussaoui @cole-miller