Skip to content
This repository was archived by the owner on Aug 28, 2024. It is now read-only.

Reorder & simplify isKeyVaultEnabled#894

Closed
jmax01 wants to merge 4 commits intomicrosoft:masterfrom
jmax01:patch-3
Closed

Reorder & simplify isKeyVaultEnabled#894
jmax01 wants to merge 4 commits intomicrosoft:masterfrom
jmax01:patch-3

Conversation

@jmax01
Copy link

@jmax01 jmax01 commented May 20, 2020

Reorder & simplify isKeyVaultEnabled to fix #867

Summary

Reorder and simplify isKeyVaultEnabled's conditionals.

Issue Type

  • Bug fixing

Starter Names

  • key vault spring boot starter

jmax01 and others added 2 commits May 20, 2020 18:05
Reorder & simplify isKeyVaultEnabled to fix microsoft#867 

## Summary
Reorder and simplify isKeyVaultEnabled's conditionals.

## Issue Type
- Bug fixing

## Starter Names
  - key vault spring boot starter
@saragluna saragluna requested a review from chenrujun May 21, 2020 06:37
@chenrujun
Copy link

I'll close this PR, Refs: Azure/azure-sdk-for-java#12134

@chenrujun chenrujun closed this Jun 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

KeyVaultEnvironmentPostProcessor.isKeyVaultEnabled should check the enabled flag before checking if AZURE_KEYVAULT_VAULT_URI is populated

3 participants

Comments