Skip to content

Commit ee967a6

Browse files
reject blank username/password secrets for gcp
1 parent e796d2e commit ee967a6

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

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)