Skip to content

fix: broken detail views and tagging for 5 services#111

Merged
DaviReisVieira merged 1 commit intomainfrom
fix/detail-views-and-tagging-110
Apr 26, 2026
Merged

fix: broken detail views and tagging for 5 services#111
DaviReisVieira merged 1 commit intomainfrom
fix/detail-views-and-tagging-110

Conversation

@DaviReisVieira
Copy link
Copy Markdown
Owner

Summary

Fixes #110

  • AppSync, Cognito-IDP, EMR — 500 on resource detail: ID field extraction picked wrong field (Name before Id, or wrong case ApiId vs apiId). Fixed via _PREFERRED_ID_FIELD overrides.
  • SES — 404 on detail: SES v1 has no single-identity describe API. Added custom handler aggregating verification + DKIM attributes.
  • API Gateway — resources not listed: registry queried apigatewayv2 (HTTP APIs) but only REST APIs (v1) existed. Added rest_apis resource type.
  • Tagging — Added tag read/write support for Cognito-IDP (via pool ARN), EMR (via ClusterId), and API Gateway REST (via ARN). SES v1 and AppSync on LocalStack don't support tagging.

Changes

File What
backend/routes/resources.py _PREFERRED_ID_FIELD entries, SES custom handler, REST API describe entry
backend/routes/stats.py rest_apis in SERVICE_REGISTRY for apigateway
backend/routes/tags.py Getter/setter functions + registry entries for 3 services
tests/test_tags_routes.py Updated supported count (21→24) and writable count (20→23)
ui/dist/ Rebuilt frontend

Test plan

  • python -m pytest tests/ -x — 288 passed
  • npx tsc -b — clean
  • Verified all 5 detail endpoints return data against LocalStack
  • Verified tag getter/setter functions with mocked boto3 clients
  • Verified API Gateway REST APIs appear in /api/stats and /api/resources/apigateway

…way and add tagging support

Fix 500/404 errors on resource detail views caused by ID field extraction
issues and missing registry entries. Add REST API support for API Gateway
(v1) alongside existing HTTP API (v2) entries. Add tag read/write support
for Cognito-IDP, ElasticMapReduce, and API Gateway REST APIs.

Closes #110
@DaviReisVieira DaviReisVieira merged commit 2b72305 into main Apr 26, 2026
2 checks passed
@DaviReisVieira DaviReisVieira deleted the fix/detail-views-and-tagging-110 branch April 26, 2026 21:13
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.

fix: broken detail views and missing tagging for AppSync, Cognito-IDP, EMR, SES, API Gateway

1 participant