Skip to content

fix: cargo fmt and CI migration step#2

Open
yusuftomilola wants to merge 2 commits into
Ibinola:feat/ct-12-ct-13-batch-revoke-and-exists-checkfrom
yusuftomilola:feat/ct-12-ct-13-batch-revoke-and-exists-check
Open

fix: cargo fmt and CI migration step#2
yusuftomilola wants to merge 2 commits into
Ibinola:feat/ct-12-ct-13-batch-revoke-and-exists-checkfrom
yusuftomilola:feat/ct-12-ct-13-batch-revoke-and-exists-check

Conversation

@yusuftomilola

Copy link
Copy Markdown

Fixes two CI failures in PR CodeGirlsInc#592:

  1. Contracts (Rust): Reorder use crate::hash_validator::HashValidator; before grouped use crate::{...} imports to satisfy cargo fmt --check
  2. Backend E2E (NestJS): Remove npm run migration:run step from CI workflow since TypeORM uses synchronize: true (auto-sync)

Ibinola and others added 2 commits June 24, 2026 16:50
…umentExistenceCheckEndpoint

- Add GET /exists/:hash — lightweight boolean existence check (CT-13)
  - Checks Redis cache first (same key as verify endpoint)
  - Falls back to Stellar verify_hash on cache miss
  - Caches positive results permanently, negative with 60s TTL
  - Validates hash with HashValidator::validate_sha256 (400 on invalid)

- Add POST /revoke/batch — concurrent batch revocation (CT-12)
  - Accepts JSON array of up to 20 RevokeRequest objects
  - Returns 400 for empty or >20 item arrays
  - Processes all revocations concurrently via join_all
  - Per-item result includes: document_hash, success, tx_hash, error
  - Partial success allowed; duplicate/already-revoked reported as errors

Closes CodeGirlsInc#544, CodeGirlsInc#545
@Ibinola Ibinola force-pushed the feat/ct-12-ct-13-batch-revoke-and-exists-check branch from 88d4fa7 to 38374e3 Compare June 26, 2026 12:48
Sign up for free to 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