Source:
<code>private void checkComment(GHIssueComment comment) throws IOException { GHUser sender = comment.getUser(); String body = comment.getBody(); <START> logger.log(Level.FINEST, "[{0}] Added comment: {1}", new Object[] { sender.getName() != null ? sender.getName() : sender.getLogin(), body }); <END> if (helper.isWhitelistPhrase(body) && helper.isAdmin(sender)) { GHIssue parent = comment.getParent(); GHUser author = parent.getUser(); if (!helper.isWhitelisted(author)) { logger.log(Level.FINEST, "Author {0} not whitelisted, adding to whitelist.", author); helper.addWhitelist(author.getLogin()); } setAccepted(true); } else if (helper.isOktotestPhrase(body) && helper.isAdmin(sender)) { logger.log(Level.FINEST, "Admin {0} gave OK to test", sender); setAccepted(true); } else if (helper.isRetestPhrase(body)) { logger.log(Level.FINEST, "Retest phrase"); if (helper.isAdmin(sender)) { logger.log(Level.FINEST, "Admin {0} gave retest phrase", sender); shouldRun = true; } else if (accepted && helper.isWhitelisted(sender)) { logger.log(Level.FINEST, "Retest accepted and user {0} is whitelisted", sender); shouldRun = true; } } else if (helper.isTriggerPhrase(body)) { logger.log(Level.FINEST, "Trigger phrase"); if (helper.isAdmin(sender)) { logger.log(Level.FINEST, "Admin {0} ran trigger phrase", sender); shouldRun = true; triggered = true; } else if (accepted && helper.isWhitelisted(sender)) { logger.log(Level.FINEST, "Trigger accepted and user {0} is whitelisted", sender); shouldRun = true; triggered = true; } } if (shouldRun) { triggerSender = sender; commentBody = body; } }</code><technical_language>Minor modification, set a variable sender.getName() is called once? check var for null get loginID if needed...</technical_language>

Target:
private void checkComment(GHIssueComment comment) throws IOException { GHUser sender = comment.getUser(); String body = comment.getBody(); String senderName = sender.getName(); logger.log(Level.FINEST, "[{0}] Added comment: {1}", new Object[] { senderName != null ? senderName : sender.getLogin(), body }); if (helper.isWhitelistPhrase(body) && helper.isAdmin(sender)) { GHIssue parent = comment.getParent(); GHUser author = parent.getUser(); if (!helper.isWhitelisted(author)) { logger.log(Level.FINEST, "Author {0} not whitelisted, adding to whitelist.", author); helper.addWhitelist(author.getLogin()); } setAccepted(true); } else if (helper.isOktotestPhrase(body) && helper.isAdmin(sender)) { logger.log(Level.FINEST, "Admin {0} gave OK to test", sender); setAccepted(true); } else if (helper.isRetestPhrase(body)) { logger.log(Level.FINEST, "Retest phrase"); if (helper.isAdmin(sender)) { logger.log(Level.FINEST, "Admin {0} gave retest phrase", sender); shouldRun = true; } else if (accepted && helper.isWhitelisted(sender)) { logger.log(Level.FINEST, "Retest accepted and user {0} is whitelisted", sender); shouldRun = true; } } else if (helper.isTriggerPhrase(body)) { logger.log(Level.FINEST, "Trigger phrase"); if (helper.isAdmin(sender)) { logger.log(Level.FINEST, "Admin {0} ran trigger phrase", sender); shouldRun = true; triggered = true; } else if (accepted && helper.isWhitelisted(sender)) { logger.log(Level.FINEST, "Trigger accepted and user {0} is whitelisted", sender); shouldRun = true; triggered = true; } } if (shouldRun) { triggerSender = sender; commentBody = body; } }