Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

maxdeichmann
Copy link
Member

@maxdeichmann maxdeichmann commented Sep 9, 2025

Important

Require admin API key for retrying background migrations, updating authentication logic and UI components.

  • Authentication:
    • AdminApiAuthService in adminApiAuth.ts refactored to separate verifyAdminAuthFromAuthString and verifyAdminAuthFromHeader.
    • New adminProcedure in trpc.ts for admin API key authentication.
  • Background Migrations:
    • retry mutation in background-migrations-router.ts now requires adminApiKey.
    • RetryBackgroundMigration component updated to include admin API key input and validation.
  • UI Changes:
    • BackgroundMigrationsTable wrapped in Page component for consistent layout.

This description was created by Ellipsis for 4467e8f. You can customize this summary. It will automatically update as commits are pushed.

Copy link

cursor bot commented Sep 9, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@maxdeichmann maxdeichmann marked this pull request as ready for review September 10, 2025 08:07
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Sep 10, 2025
@maxdeichmann maxdeichmann changed the title Require admin api key for background migration retries fix: require admin api key for background migration retries in OSS Sep 10, 2025
@maxdeichmann maxdeichmann added this pull request to the merge queue Sep 10, 2025
@dosubot dosubot bot added the auto-merge This PR is set to be merged label Sep 10, 2025
Merged via the queue into main with commit a618d53 Sep 10, 2025
30 checks passed
@maxdeichmann maxdeichmann deleted the cursor/LFE-6632-require-admin-api-key-for-background-migration-retries-1b59 branch September 10, 2025 09:24
@dosubot dosubot bot removed the auto-merge This PR is set to be merged label Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat-api-auth size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants