Skip to content

Commit

Permalink
Merge pull request #16021 from vsevel/vault_vertx_illegal_state
Browse files Browse the repository at this point in the history
Deactivate Vault Config Source refresh if running in non blocking thread
  • Loading branch information
sberyozkin authored Mar 26, 2021
2 parents 5207b18 + 70d565c commit 13c5469
Showing 1 changed file with 7 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
import static java.util.Collections.emptyMap;
import static java.util.stream.Collectors.toMap;

import java.util.Collections;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
Expand All @@ -14,6 +15,7 @@

import io.quarkus.arc.Arc;
import io.quarkus.vault.VaultKVSecretEngine;
import io.smallrye.mutiny.infrastructure.Infrastructure;

public class VaultConfigSource implements ConfigSource {

Expand Down Expand Up @@ -58,6 +60,11 @@ private Map<String, String> getSecretConfig() {
return cacheEntry.getValue();
}

if (!Infrastructure.canCallerThreadBeBlocked()) {
// running in a non blocking thread, best effort to return cached values if any
return cacheEntry != null ? cacheEntry.getValue() : Collections.emptyMap();
}

Map<String, String> properties = new HashMap<>();

try {
Expand Down

0 comments on commit 13c5469

Please sign in to comment.