awslabs / awslabs/llrt

Reduce number of dependencies of llrt_crypto

Open
#877 3 comments 2 reactions 0 assignees View on GitHub
dependencies enhancement help wanted
Dominant language
Rust
Stars
8.8k
Forks
393
Avg merge
1d 11h
Merged PRs (30d)
45

Description

The module has too many dependencies, it really pollutes any project trying to import it since some have very deep web of dependencies. For example we pull in two different bigint implementations (`num-bigint-dig` and `crypto-bigint`)
We have a lot of duplicate with ring anyway and we should prefer using it. Curve crypto is still not in ring, but planned (https://github.com/briansmith/ring/issues/1771)

Contributor guide

Open the contributing guide

Research direction

Start by inspecting the llrt_crypto dependency declarations and generating its dependency tree to identify duplicate bigint implementations and overlap with ring. Review which curve-cryptography features still require non-ring dependencies; done means reducing redundant dependencies without removing required curve support.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
cryptography
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.