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

Bump dependencies + clippy warnings #71

Merged
merged 5 commits into from
Sep 11, 2023
Merged

Bump dependencies + clippy warnings #71

merged 5 commits into from
Sep 11, 2023

Conversation

r3dlight
Copy link
Contributor

@r3dlight r3dlight commented Sep 7, 2023

  • Bump ssh-key & ed25519-dalek dependencies (security fix)
  • Bump rsa, signature, strum-macros, strum and ring crates
  • Fix some really quick clippy warnings

@HsuJv
Copy link
Collaborator

HsuJv commented Sep 7, 2023

Hi, I don't know if it's convenient for you to fix these workflow errors, if not I can take a look at them this weekend.

@r3dlight
Copy link
Contributor Author

r3dlight commented Sep 8, 2023

Hum not sure I can fix these two "public_key" and "key_exchange" mutability warnings without breaking more things...

@HsuJv
Copy link
Collaborator

HsuJv commented Sep 8, 2023

Never mind, I can take a look this weekend. :-)

@r3dlight
Copy link
Contributor Author

r3dlight commented Sep 8, 2023

Great thank you :-)

@HsuJv
Copy link
Collaborator

HsuJv commented Sep 10, 2023

Hi @r3dlight ,

I've done with the workflow fixes and also bumped the aes crate to 0.8
Please process the PR in your repo so that I can merge this PR.

BRs.

* fix workflows

* bump aes to 0.8, fix workflows
@r3dlight
Copy link
Contributor Author

r3dlight commented Sep 10, 2023

Hi HsuJv,

Great, merged, thanks for the patch :)
BRs

@HsuJv HsuJv merged commit 661dbac into 1148118271:main Sep 11, 2023
6 checks passed
@HsuJv
Copy link
Collaborator

HsuJv commented Sep 11, 2023

This closes #70

@HsuJv HsuJv mentioned this pull request Sep 11, 2023
@HsuJv
Copy link
Collaborator

HsuJv commented Sep 11, 2023

v0.3.3 has been published

@HsuJv HsuJv mentioned this pull request Sep 16, 2023
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