decentraland / asset-bundle-registry / 25111749148

71%

push

github

feat: add denylist CRUD endpoints with moderator authorization (#114)

* feat: add denylist table, migrations, and CRUD endpoints with moderator authorization

- Migration 1745846400000_denylist.ts: new denylist table with entity_id (PK),
  reason, created_by, created_at, updated_at audit columns and case-insensitive indexes
- Extends IDbComponent with getDenylist / addDenylistEntry / removeDenylistEntry
- Extends IRefreshableFeaturesComponent with getUserModerators() polling
  dapps-platform_user_moderators feature flag (same pattern as malicious-profiles)
- GET /denylist: public endpoint returning all blocked entity IDs
- POST /denylist/:entityId: signed-fetch + moderator-only, upserts an entry
- DELETE /denylist/:entityId: signed-fetch + moderator-only, removes an entry
- Unit and integration tests covering all paths including 403 for non-moderators

* fix: address PR feedback on denylist feature

- Wrap long line in refreshable-features/component.ts (prettier)
- Preserve original created_by on upsert conflict
- Remove redundant LOWER(entity_id) index (data already stored lowercase)

* docs: add denylist endpoints to OpenAPI spec

* feat: filter denylisted entities from POST /entities/active (#115)

* feat: filter denylisted entities from POST /entities/active

Mirrors the catalyst behavior by excluding denylisted entities at the
SQL level using a NOT EXISTS subquery against the denylist table, avoiding
a separate round-trip to fetch the full denylist.

* fix: drop redundant LOWER on denylist side and fix flaky ordering test

- Use denylist.entity_id = LOWER(registries.id) instead of comparing
  both sides with LOWER(), since addDenylistEntry already stores entity_id
  in lowercase — allows PostgreSQL to use the PK index on entity_id
- Make "should return multiple entities" order-independent with
  arrayContaining since getSortedRegistriesByPointers has no guaranteed
  ordering when sortOrder is not set

609 of 896 branches covered (67.97%)

Branch coverage included in aggregate %.

56 of 68 new or added lines in 7 files covered. (82.35%)

1599 of 2213 relevant lines covered (72.25%)

37.09 hits per line