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

Update rsa #1152

Merged
merged 1 commit into from Apr 13, 2023
Merged

Update rsa #1152

merged 1 commit into from Apr 13, 2023

Conversation

yubiuser
Copy link
Contributor

@yubiuser yubiuser commented Apr 12, 2023

This is one of a series of PRs that will update dependencies not included in #1140.

This one updates rsa from v0.6 to v0.8.2
Breaking change were

@yubiuser yubiuser force-pushed the update/rsa branch 12 times, most recently from 511fcad to 736473d Compare April 12, 2023 20:17
@yubiuser yubiuser marked this pull request as ready for review April 12, 2023 20:19
Signed-off-by: Christian König <ckoenig@posteo.de>
@roderickvd
Copy link
Member

Good work, that crypto API has been very unstable over the past releases, and usually not easy to migrate. Glad that we can remove that unsafe marker.

@roderickvd roderickvd merged commit 2718d22 into librespot-org:dev Apr 13, 2023
9 checks passed
@yubiuser yubiuser deleted the update/rsa branch April 13, 2023 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants