Skip to content
This repository has been archived by the owner on Oct 11, 2018. It is now read-only.

Commit

Permalink
Merge pull request #3 from rishabh9/npe-fix
Browse files Browse the repository at this point in the history
Fix Null pointer exception in loggers.
  • Loading branch information
rishabh9 authored Mar 15, 2017
2 parents 3d0c1d4 + e70a0d5 commit 8602799
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions app/controllers/helpers/MailchimpKeyVerifier.java
Original file line number Diff line number Diff line change
Expand Up @@ -62,12 +62,12 @@ Optional<Result> verifyAndCompleteInstallation(Installation installation, Messag
String apiKey = meta[0];
String dataCentre = meta[1];
if (StringUtils.hasText(apiKey) && StringUtils.hasText(dataCentre)) {
log.info("Verifying API key {} for installation {}", apiKey, installation.getId().toHexString());
log.info("Verifying API key {} for installation {}", apiKey, installation.getInstallationId());
try {
WSResponse response = ws.url(String.format(appConfig.getString(Constants.MAILCHIMP_VERIFY_URL), dataCentre))
.setAuth("username", apiKey).get().toCompletableFuture().get();
if (response.getStatus() == Http.Status.OK) {
log.info("Mailchimp API key is verified for installation {}", installation.getId().toHexString());
log.info("Mailchimp API key is verified for installation {}", installation.getInstallationId());
return Optional.empty();
} else {
log.info("The provided Mailchimp API key doesn't seems to be valid one. Installation {}",
Expand Down

0 comments on commit 8602799

Please sign in to comment.