Skip to content

feat: Migrate from FreshCryptoLib to SmoothCryptoLib - #76

Open
itasli wants to merge 2 commits into
base:mainfrom
itasli:dev
Open

feat: Migrate from FreshCryptoLib to SmoothCryptoLib#76
itasli wants to merge 2 commits into
base:mainfrom
itasli:dev

Conversation

@itasli

Copy link
Copy Markdown

Replace deprecated and unaudited FreshCryptoLib dependency with SmoothCryptoLib from same developpers, improves blockchain compatibility as well.

Changes

  • Remove: FreshCryptoLib submodule and imports
  • Add: SmoothCryptoLib submodule
  • Update: Base64Url → OpenZeppelin Base64.encodeURL()
  • Maintain: Identical API compatibility (no breaking changes)

Benefits

  • Compatibility: Works on Polygon zkEVM and other chains without ModExp precompile
  • Audits: Multiple security audits + partial formal verification

Compatibility

  • API: Identical function signatures
  • Tests: All existing tests pass
  • Fallback: Maintains RIP-7212 precompile + library pattern

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

Sign up for freeto 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

@itasli@cb-heimdall