diff --git a/nifi-extension-bundles/nifi-github-bundle/nifi-github-extensions/src/main/java/org/apache/nifi/github/GitHubRepositoryClient.java b/nifi-extension-bundles/nifi-github-bundle/nifi-github-extensions/src/main/java/org/apache/nifi/github/GitHubRepositoryClient.java index 3c1af3a2c498..ca20991d8c1b 100644 --- a/nifi-extension-bundles/nifi-github-bundle/nifi-github-extensions/src/main/java/org/apache/nifi/github/GitHubRepositoryClient.java +++ b/nifi-extension-bundles/nifi-github-bundle/nifi-github-extensions/src/main/java/org/apache/nifi/github/GitHubRepositoryClient.java @@ -474,18 +474,22 @@ private void throwPathOrBranchNotFound(final FileNotFoundException fileNotFoundE private GitCommit toGitCommit(final GHCommit ghCommit) throws IOException { GitCommit commit = commitCache.getIfPresent(ghCommit.getSHA1()); - if (commit != null) { - return commit; - } else { + + if (commit == null) { final GHCommit.ShortInfo shortInfo = ghCommit.getCommitShortInfo(); + final String author = ghCommit.getAuthor() != null + ? ghCommit.getAuthor().getLogin() + : shortInfo.getAuthor().getName(); + commit = new GitCommit( - ghCommit.getSHA1(), - ghCommit.getAuthor().getLogin(), - shortInfo.getMessage(), - Instant.ofEpochMilli(shortInfo.getCommitDate().getTime())); + ghCommit.getSHA1(), + author, + shortInfo.getMessage(), + Instant.ofEpochMilli(shortInfo.getCommitDate().getTime())); commitCache.put(ghCommit.getSHA1(), commit); - return commit; } + + return commit; } private T execute(final GHRequest action) throws FlowRegistryException, IOException {