Skip to content
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

Feature(Backend): Secret Syncs #2952

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

scott-ray-wilson
Copy link
Contributor

@scott-ray-wilson scott-ray-wilson commented Jan 8, 2025

Description 📣

This PR contains the proposed backend for Secret Syncs (Integrations v2). Secret Syncs utilize App Connections to sync, import, and erase secrets from third-party services. AWS Parameter Store with minimal configuration has been used to demonstrate functionality and backend/data structure.

Also includes minor refactoring/additions to App Connections and Org Permissions. App Connections now have Connect permission with conditions.

enable appConnections feature flag to preview.

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

Tests 🛠️

# Here's some code block to paste some code snippets

backend/src/db/migrations/20241219210911_secret-sync.ts Outdated Show resolved Hide resolved
backend/src/db/migrations/20241219210911_secret-sync.ts Outdated Show resolved Hide resolved
backend/src/db/migrations/20241219210911_secret-sync.ts Outdated Show resolved Hide resolved
backend/src/server/routes/v1/index.ts Show resolved Hide resolved
db.ref("updatedAt").withSchema(TableName.AppConnection).as("connectionUpdatedAt")
);

const appendTableName = (filterObj: object): SecretSyncFindFilter =>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Careful!! I feel this may cause more issues. Just a note

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left comments, will make generic at some point down the road!

@akhilmhdh
Copy link
Member

Need to do application testing - But i guess don't have the UI in this one. I'll read the sync part once again later.

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.

2 participants