Merge pull request #23642 from gaborando/master

P-Limit Deadlock Solved
This commit is contained in:
Ben Lambert
2024-03-19 10:59:04 +01:00
committed by GitHub
2 changed files with 8 additions and 5 deletions
+5
View File
@@ -0,0 +1,5 @@
---
'@backstage/plugin-vault-backend': patch
---
Removed lock inside recursion to avoid deadlocks
@@ -147,11 +147,9 @@ export class VaultClient implements VaultApi {
result.data.keys.map(async secret => {
if (secret.endsWith('/')) {
secrets.push(
...(await this.limit(() =>
this.listSecrets(`${secretPath}/${secret.slice(0, -1)}`, {
secretEngine: mount,
}),
)),
...(await this.listSecrets(`${secretPath}/${secret.slice(0, -1)}`, {
secretEngine: mount,
})),
);
} else {
const vaultUrl =