@shamsharoonchecks…feat/api-key-ip-allowlist-table → production7 files · +398 −16review: @abir-taheerreview: @anshugarg15updated 2w agoproject_api_key_allowed_ips join table (v99 migration) to extract IP allowlisting out of the project_api_keys.allowed_ips TEXT[] columnpackages/db/data-migrations/2025_05_27_backfill_api_key_allowed_ips.tsallowed_ips column + remove dual-write codepackages/db/prisma/schema.prisma — new ApiKeyAllowedIp model + relationpackages/db/liquibase/migrations/v99_.../ — up/down migrationpackages/db/liquibase/changelog/changelog-root.json — register v99packages/db/data-migrations/2025_05_27_backfill_...ts — backfill scriptapps/apollo/src/lib/apiKeys/dbUtils/index.ts — dual-write logic, resolveAllowedIps helpercreateAPIKey with allowedIps writes to both columnsupdateAllowedIps writes to both columns in a transactiongetAPIKeyInfo / getAllAPIKeysForProject return IPs from join table (fallback to old column)Based on git blame analysis of 7 file(s):
| Contributor | Contribution | Files |
|---|---|---|
| shams haroon | 22% | 7 |
| Himanshu Dixit | 12% | 3 |
| anshugarg15 | 11% | 3 |
| sarahsimionescu | 10% | 2 |
| Rahul Tarak | 8% | 3 |
Based on git history, shams haroon or Himanshu Dixit would be good reviewer(s) for this PR.
🤖 Based on git blame with recency weighting (recent edits count more).