Skip to content

Commit b5e421c

Browse files
authored
Merge pull request #1362 from obiba/fix/indexation
MLSTRM-441
2 parents 344531a + fb9f355 commit b5e421c

File tree

1 file changed

+11
-3
lines changed

1 file changed

+11
-3
lines changed

mica-core/src/main/java/org/obiba/mica/core/service/AbstractGitPersistableService.java

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import com.google.common.collect.Lists;
2727
import org.apache.commons.math3.util.Pair;
2828
import org.apache.shiro.SecurityUtils;
29+
import org.apache.shiro.session.UnknownSessionException;
2930
import org.apache.shiro.subject.Subject;
3031
import org.joda.time.DateTime;
3132
import org.obiba.git.CommitInfo;
@@ -194,8 +195,15 @@ public T updateStatus(String id, RevisionStatus status) {
194195

195196
protected String getCurrentUsername() {
196197
Subject subject = SecurityUtils.getSubject();
197-
return subject == null || subject.getPrincipal() == null
198-
? AbstractGitWriteCommand.DEFAULT_AUTHOR_NAME
199-
: subject.getPrincipal().toString();
198+
return extractAuthorName(subject, AbstractGitWriteCommand.DEFAULT_AUTHOR_NAME);
199+
}
200+
201+
private String extractAuthorName(Subject subject, String defaultName) {
202+
try {
203+
if (subject != null && subject.getPrincipal() != null)
204+
return subject.getPrincipal().toString();
205+
} catch (UnknownSessionException ignore) {
206+
}
207+
return defaultName;
200208
}
201209
}

0 commit comments

Comments
 (0)