Skip to content

Conversation

wwillard7800
Copy link
Contributor

DAT-17324

Impact

  • Bug fix (non-breaking change which fixes expected existing functionality)
  • Enhancement/New feature (adds functionality without impacting existing logic)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

Things to worry about

Additional Context

@@ -1040,6 +1085,18 @@
return configurationProperties;
}

private void handleVaultProperties(Map<String, Object> scopeValues) {
if (StringUtil.isNotEmpty(this.vaultAddr)) {

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation Note

Invoking
StringUtil.isNotEmpty
should be avoided because it has been deprecated.
if (StringUtil.isNotEmpty(this.vaultAddr)) {
scopeValues.put("liquibase.vault.addr", this.vaultAddr);
}
if (StringUtil.isNotEmpty(this.vaultToken)) {

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation Note

Invoking
StringUtil.isNotEmpty
should be avoided because it has been deprecated.
if (StringUtil.isNotEmpty(this.vaultToken)) {
scopeValues.put("liquibase.vault.token", this.vaultToken);
}
if (StringUtil.isNotEmpty(this.vaultNamespace)) {

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation Note

Invoking
StringUtil.isNotEmpty
should be avoided because it has been deprecated.
@suryaaki2 suryaaki2 merged commit d1e07da into master Sep 4, 2024
44 of 45 checks passed
@suryaaki2 suryaaki2 deleted the DAT-17324 branch September 4, 2024 15:39
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