Skip to content

Commit c705447

Browse files
reject blank username/password secrets for gcp (#230)
1 parent e796d2e commit c705447

2 files changed

Lines changed: 11 additions & 1 deletion

File tree

ojdbc-provider-gcp/src/main/java/oracle/jdbc/provider/gcp/resource/GcpSecretManagerPasswordProvider.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,11 @@ public GcpSecretManagerPasswordProvider() {
6363
public char[] getPassword(Map<Parameter, CharSequence> parameterValues) {
6464
ByteString secret = getSecret(parameterValues);
6565
String password = secret.toStringUtf8();
66+
67+
if (password == null || password.trim().isEmpty()) {
68+
throw new IllegalArgumentException("Password secret content is blank.");
69+
}
70+
6671
return password.toCharArray();
6772
}
6873

ojdbc-provider-gcp/src/main/java/oracle/jdbc/provider/gcp/resource/GcpSecretManagerUsernameProvider.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,12 @@ public GcpSecretManagerUsernameProvider() {
6060

6161
@Override
6262
public String getUsername(Map<Parameter, CharSequence> parameterValues) {
63-
return getSecret(parameterValues).toStringUtf8();
63+
String username = getSecret(parameterValues).toStringUtf8();
64+
65+
if (username == null || username.trim().isEmpty()) {
66+
throw new IllegalArgumentException("Username secret content is blank");
67+
}
68+
return username;
6469
}
6570

6671
}

0 commit comments

Comments
 (0)