Skip to content

Conversation

@macwilam
Copy link

new branch that makes initial commit to ruse aws_lc_rs instead of openssl. This make the dependency simpler. This is a initial commit that tries to make a rewrite of existing functionality. md5 and des were not ported to this feature as they cannot be done within aws_lc_rs.

Important notes:

  • I have done it as a simple rewrite - no design decisions or anything like it I just tried to replicate functionality 1:1
  • I am not cryptography expert nor snmp expert. I think I was able to make this PR as a starting point proof of concept.
  • No MD5 and DES as they are not in aws_lc_rs. Also if I understand correctly I cannot have them If we want fips. It can be added but with other dependencies
  • I did use Claude Opus 4.5 to help myself make the work faster.
  • It was not vibe coded - I reviewed everything and tested this with my program that uses this repo code after changes

…nssl. This make the dependency simpler. This is a initial commit that tries to make a rewrite of existing functionality. md5 and des were not ported to this feater as they cannot be done within aws_lc_rs.
@macwilam macwilam mentioned this pull request Dec 10, 2025
@divi255
Copy link
Contributor

divi255 commented Dec 12, 2025

thanks! I'll come to this a bit later as switching to rustls will require minor version bump anyway. expect this weekend or next week

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