Clean up integration credentials storage infra #8540
Assignees
Labels
platform/backend
AutoGPT Platform - Back end
Comments
3 tasks
github-merge-queue bot
pushed a commit
that referenced
this issue
Nov 29, 2024
…kend (#8648) - Move `autogpt_libs.supabase_integration_credentials_store` into `backend` - `.store` -> `backend.integrations.credentials_store` - `.types` -> added to `backend.data.model` - Rename `SupabaseIntegrationCredentialsStore` to `IntegrationCredentialsStore` We wanted to get a few security things in quickly in #8403 and had to make some compromises to do so. This picks those up and fixes them. - Resolves #8540 ### Checklist 📋 #### For code changes: - [x] I have clearly listed my changes in the PR description - [x] I have made a test plan - [x] I have tested my changes according to the test plan: <!-- Put your test plan here: --> --------- Co-authored-by: Reinier van der Leer <[email protected]> Co-authored-by: Aarushi <[email protected]>
aarushik93
added a commit
that referenced
this issue
Dec 1, 2024
…kend (#8648) - Move `autogpt_libs.supabase_integration_credentials_store` into `backend` - `.store` -> `backend.integrations.credentials_store` - `.types` -> added to `backend.data.model` - Rename `SupabaseIntegrationCredentialsStore` to `IntegrationCredentialsStore` We wanted to get a few security things in quickly in #8403 and had to make some compromises to do so. This picks those up and fixes them. - Resolves #8540 ### Checklist 📋 #### For code changes: - [x] I have clearly listed my changes in the PR description - [x] I have made a test plan - [x] I have tested my changes according to the test plan: <!-- Put your test plan here: --> --------- Co-authored-by: Reinier van der Leer <[email protected]> Co-authored-by: Aarushi <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A few updates were made in the past weeks which have left some technical debt, including:
TODO
ENCRYPTION_KEY. Fernet has very specific requirements for what it accepts as an encryption key.SupabaseIntegrationCredentialsStoreandIntegrationCredentialsManagerlocking mechanisms (see comment)SupabaseIntegrationCredentialsStoretoIntegrationCredentialsStoreautogpt_libs.supabase_integration_credentials_storetointegration_credentials_stroebackendinautogpt_libs. This will probably involve movingIntegrationCredentialsStoretobackend.The text was updated successfully, but these errors were encountered: