Skip to content

fix: keepersecurity support for shortcuts#4825

Merged
Skarlso merged 3 commits intoexternal-secrets:mainfrom
pepordev:fix/keepersecurity-support-shortcuts
Jun 4, 2025
Merged

fix: keepersecurity support for shortcuts#4825
Skarlso merged 3 commits intoexternal-secrets:mainfrom
pepordev:fix/keepersecurity-support-shortcuts

Conversation

@pepordev
Copy link
Copy Markdown
Contributor

Problem Statement

Add support for shortcuts in keeper security provider

Related Issue

Fixes #4600

Checklist

  • I have read the contribution guidelines
  • All commits are signed with git commit --signoff
  • My changes have reasonable test coverage
  • All tests pass with make test
  • I ensured my PR is ready for review with make reviewable

@pepordev pepordev requested a review from a team as a code owner May 22, 2025 08:49
@pepordev pepordev requested a review from moolen May 22, 2025 08:49
Signed-off-by: Pedro Parra Ortega <pedro.parraortega@enreach.com>
@pepordev pepordev force-pushed the fix/keepersecurity-support-shortcuts branch from b09b19a to 86f5da3 Compare May 22, 2025 08:50
@Skarlso Skarlso merged commit d6ecbf4 into external-secrets:main Jun 4, 2025
1 check passed
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Jun 4, 2025

Skarlso added a commit that referenced this pull request Jun 4, 2025
Skarlso added a commit that referenced this pull request Jun 4, 2025
This reverts commit d6ecbf4.

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
Skarlso added a commit that referenced this pull request Jun 4, 2025
This reverts commit d6ecbf4.

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
@Skarlso
Copy link
Copy Markdown
Contributor

Skarlso commented Jun 4, 2025

@ppodevlabs Sorry, I had to revert this. It was causing unit test failures. :/ Please open another PR.

pepordev pushed a commit to pepordev/external-secrets that referenced this pull request Jun 11, 2025
)" (external-secrets#4870)

This reverts commit d6ecbf4.

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
Signed-off-by: Pedro Parra Ortega <pedro.parraortega@enreach.com>
alliseeisgold pushed a commit to alliseeisgold/external-secrets that referenced this pull request Jul 10, 2025
Signed-off-by: Pedro Parra Ortega <pedro.parraortega@enreach.com>
Co-authored-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
Signed-off-by: asrormirzoev <asrormirzoev@yandex-team.ru>
alliseeisgold pushed a commit to alliseeisgold/external-secrets that referenced this pull request Jul 10, 2025
)" (external-secrets#4870)

This reverts commit d6ecbf4.

Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>
Signed-off-by: asrormirzoev <asrormirzoev@yandex-team.ru>
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.

Keeper provider is failing with more than 1 secret found

3 participants