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

Add new secret sync destination fields #2150

Merged
merged 14 commits into from
Mar 4, 2024

Conversation

Zlaticanin
Copy link
Contributor

@Zlaticanin Zlaticanin commented Feb 20, 2024

This PR includes adding new fields to AWS, GCP and GH sync destinations. As well as adding a new endpoint for github-apps

@github-actions github-actions bot added size/L and removed size/XS labels Feb 22, 2024
@Zlaticanin Zlaticanin changed the title Add project_id param to the GCP sync dest Add new secret sync destination fields Feb 22, 2024
@github-actions github-actions bot added size/XL and removed size/L labels Feb 22, 2024
@fairclothjm fairclothjm added this to the 3.26.0 milestone Feb 22, 2024
Copy link
Contributor

@raymonstah raymonstah left a comment

Choose a reason for hiding this comment

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

Just want to point out that /github-apps is not a destination. It's a configuration endpoint that allows users creating a new github destination to reference the application as an alternative to passing in an access_token. This makes access_token an optional field now.

internal/sync/secrets_sync.go Outdated Show resolved Hide resolved
vault/provider.go Outdated Show resolved Hide resolved
vault/resource_secrets_sync_aws_destination_test.go Outdated Show resolved Hide resolved
vault/resource_secrets_sync_aws_destination_test.go Outdated Show resolved Hide resolved
vault/resource_secrets_sync_gh_apps_destination.go Outdated Show resolved Hide resolved
website/docs/r/secrets_sync_gh_apps_destination.html.md Outdated Show resolved Hide resolved
website/docs/r/secrets_sync_gh_apps_destination.html.md Outdated Show resolved Hide resolved
website/docs/r/secrets_sync_gh_apps_destination.html.md Outdated Show resolved Hide resolved
website/docs/r/secrets_sync_gh_apps_destination.html.md Outdated Show resolved Hide resolved
website/docs/r/secrets_sync_gh_apps_destination.html.md Outdated Show resolved Hide resolved
@fairclothjm fairclothjm modified the milestones: 3.26.0, 3.27.0 Feb 28, 2024
@Zlaticanin Zlaticanin requested a review from raymonstah March 4, 2024 22:50
Copy link
Contributor

@raymonstah raymonstah left a comment

Choose a reason for hiding this comment

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

🎉

@Zlaticanin Zlaticanin merged commit cb4aab4 into main Mar 4, 2024
3 checks passed
@Zlaticanin Zlaticanin deleted the VAULT-24263/add-project-id-param-gcp branch March 4, 2024 23:29
@fairclothjm fairclothjm modified the milestones: 3.27.0, 4.0.0 Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants