Skip to content

Commit 5bb8c8b

Browse files
committed
CYBS-737: Typo refactor
1 parent 994dcb1 commit 5bb8c8b

2 files changed

Lines changed: 11 additions & 13 deletions

File tree

java/src/main/java/com/cybersource/ws/client/Identity.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -68,14 +68,14 @@ private void setupJdkServerCerts() throws SignException {
6868
if (x509Cert != null) {
6969
String subjectDN = x509Cert.getSubjectDN().getName();
7070
if (subjectDN != null) {
71-
String[] subjectDNrray = subjectDN.split("SERIALNUMBER=");
72-
if (subjectDNrray.length == 1 && subjectDNrray[0].toLowerCase().contains(CYBS_CERT_AUTH.toLowerCase())){
73-
name = keyAlias = subjectDNrray[0].split("=")[1];
71+
String[] subjectDNArray = subjectDN.split("SERIALNUMBER=");
72+
if (subjectDNArray.length == 1 && subjectDNArray[0].toLowerCase().contains(CYBS_CERT_AUTH.toLowerCase())){
73+
name = keyAlias = subjectDNArray[0].split("=")[1];
7474
}
75-
else if (subjectDNrray.length == 2 && subjectDNrray[1].toLowerCase().contains(SERVER_ALIAS.toLowerCase())) {
76-
String subjectDName = subjectDNrray[1].split("=")[1];
75+
else if (subjectDNArray.length == 2 && subjectDNArray[1].toLowerCase().contains(SERVER_ALIAS.toLowerCase())) {
76+
String subjectDName = subjectDNArray[1].split("=")[1];
7777
name = subjectDName.substring(0, subjectDName.length()-1);
78-
serialNumber = subjectDNrray[1];
78+
serialNumber = subjectDNArray[1];
7979
keyAlias = "serialNumber=" + serialNumber + ",CN=" + name;
8080
}else{
8181
throw new SignException("Exception while obtaining private key from KeyStore with alias, '" + merchantConfig.getKeyAlias() + "'");

java/src/main/java/com/cybersource/ws/client/SecurityUtil.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -357,8 +357,6 @@ private static void loadJavaKeystore(MerchantConfig merchantConfig, Logger logge
357357
KeyStore keystore = KeyStore.getInstance(KeyStore.getDefaultType());
358358
keystore.load(is, merchantConfig.getCacertPassword().toCharArray());
359359

360-
Identity identity;
361-
362360
java.security.cert.Certificate[] cert = keystore.getCertificateChain(merchantConfig.getKeyAlias());
363361
if (cert == null) {
364362
throw new SignException("Empty Keystore or Missing Certificate ");
@@ -372,14 +370,14 @@ private static void loadJavaKeystore(MerchantConfig merchantConfig, Logger logge
372370
+ merchantConfig.getKeyAlias() + "'");
373371
throw new SignException(e);
374372
}
375-
376-
for (int i = 0; i < cert.length; i++) {
377-
if (merchantConfig.getKeyAlias().equals(keystore.getCertificateAlias(cert[i]))) {
378-
identity = new Identity(merchantConfig, (X509Certificate) cert[i], key, logger);
373+
Identity identity;
374+
for (java.security.cert.Certificate certificate : cert) {
375+
if (merchantConfig.getKeyAlias().equals(keystore.getCertificateAlias(certificate))) {
376+
identity = new Identity(merchantConfig, (X509Certificate) certificate, key, logger);
379377
localKeyStoreHandler.addIdentityToKeyStore(identity, logger);
380378
identities.put(identity.getKeyAlias(), identity);
381379
} else {
382-
identity = new Identity(merchantConfig, (X509Certificate) cert[i], logger);
380+
identity = new Identity(merchantConfig, (X509Certificate) certificate, logger);
383381
localKeyStoreHandler.addIdentityToKeyStore(identity, logger);
384382
identities.put(identity.getName(), identity);
385383
}

0 commit comments

Comments
 (0)