search

Layr-Labs / eigenda-proxy

Secure and optimized communication layer for rollups using EigenDA.
MIT License
15 stars 22 forks source link

Fix s3 panic from keccak commitment put requests #129

Closed samlaf closed 2 months ago

samlaf commented 2 months ago

Fixes Issue

Fixes #128

Changes proposed

2 main things in this PR:

  1. add a test to reproduce the panic
  2. fix the test (https://github.com/Layr-Labs/eigenda-proxy/commit/d8c605b980b43a63559c278836afbe58de66910a)

Screenshots (Optional)

Note to reviewers

I'm fine with using either this fix or the one in #125
But if we want the one in #125 then we should still include the test from this PR.