Skip to content

Refactor: api key creation/retrieval/deletion to use infra api #68

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

ben-fornefeld
Copy link
Member

@ben-fornefeld ben-fornefeld commented May 23, 2025

To support secret hashing, the dashboard should rely on the infra api to handle api-keys. This pr refactors the key management to use the dedicated infra apis instead of direct db access.

Also completes E2B-2352

@ben-fornefeld ben-fornefeld requested a review from dobrac May 23, 2025 14:09
@ben-fornefeld ben-fornefeld self-assigned this May 23, 2025
Copy link

linear bot commented May 23, 2025

Copy link

vercel bot commented May 23, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2025 4:25pm
web-juliett ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2025 4:25pm

@ben-fornefeld ben-fornefeld marked this pull request as draft May 23, 2025 14:12
@ben-fornefeld ben-fornefeld force-pushed the use-crud-endpoints-for-api-keys-access-tokens-in-dashboard-e2b-1588 branch from cd273a5 to 695efb3 Compare May 23, 2025 14:15
@ben-fornefeld ben-fornefeld marked this pull request as ready for review May 23, 2025 14:34
@e2b-dev e2b-dev deleted a comment from linear bot May 23, 2025
Co-authored-by: Jakub Dobry <jakub.dobry8@gmail.com>
@ben-fornefeld ben-fornefeld merged commit 839abed into main May 26, 2025
4 checks passed
@ben-fornefeld ben-fornefeld deleted the use-crud-endpoints-for-api-keys-access-tokens-in-dashboard-e2b-1588 branch May 26, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants