alextzimas/sew-20-migrate-the-blocklist-from-vercel-edge-config-to-redis - Branch - MystenLabs/walrus-sites

feat: support a RedisBlocklistChecker and improve code structure

#371Merged

Comparing

alextzimas/sew-20-migrate-the-blocklist-from-vercel-edge-config-to-redis

(

badab17

) with

main

(

efb2744

)

Untouched: 3

Benchmarks

Passed

decompress plaintext encoding

portal/common/lib/decompress_data.bench.ts::decompressData::decompress plaintext encoding

16.1 µs16.1 µs

decompress gzip encoding

portal/common/lib/decompress_data.bench.ts::decompressData::decompress gzip encoding

253.6 µs254.8 µs

decompress deflate encoding

portal/common/lib/decompress_data.bench.ts::decompressData::decompress deflate encoding

-1%

248 µs249.4 µs

Commits

Click on a commit to change the comparison range

Base

main

efb2744

+0.07%

Improve docstrings

926077e

8 months ago

by Tzal3x

-0.03%

Deduce storage variant from environment variables

a534733

8 months ago

by Tzal3x

+0.1%

Fix instanstiateBlocklistChecker

a2dc633

8 months ago

by Tzal3x

+0.03%

Fix configuration_loader logic

dff8e4c

8 months ago

by Tzal3x

+0.03%

refactor: use object literal mapping instead of switch This is better than a non-exhaustive switch statement. Plus it follows the Open/Closed Principle - extend by adding to map, not modifying code. The main tradeoff is that it's slightly more abstract than a switch statement. It becomes more valuable as we add more variants.

0c98cf7

8 months ago

by Tzal3x

-0.43%

Use RedisClient.isReady instead of custom `connected` field

badab17

8 months ago

by Tzal3x

Home Terms Privacy Docs