Skip to content

Commit 2126078

Browse files
Polish
Closes gh-2917
1 parent 44a6da4 commit 2126078

File tree

1 file changed

+10
-7
lines changed

1 file changed

+10
-7
lines changed

spring-session-data-redis/src/main/java/org/springframework/session/data/redis/ReactiveRedisIndexedSessionRepository.java

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -385,10 +385,10 @@ private Mono<RedisSession> getSession(String sessionId, boolean allowExpired) {
385385

386386
@Override
387387
public Mono<Void> deleteById(String id) {
388-
return internalDeleteById(id).then();
388+
return deleteAndReturn(id).then();
389389
}
390390

391-
public Mono<Session> internalDeleteById(String id) {
391+
private Mono<RedisSession> deleteAndReturn(String id) {
392392
// @formatter:off
393393
return getSession(id, true)
394394
.flatMap((session) -> this.sessionRedisOperations.delete(getExpiredKey(session.getId()))
@@ -438,11 +438,13 @@ private Mono<Void> onSessionCreatedChannelMessage(ReactiveSubscription.Message<S
438438
}
439439

440440
private Mono<Void> onKeyDestroyedMessage(ReactiveSubscription.Message<String, String> message) {
441-
return Mono.just(message.getMessage()).filter((key) -> key.startsWith(getExpiredKeyPrefix())).map((key) -> {
442-
int sessionIdBeginIndex = key.lastIndexOf(":") + 1;
443-
return key.substring(sessionIdBeginIndex);
444-
})
445-
.flatMap(this::internalDeleteById)
441+
// @formatter:off
442+
return Mono.just(message.getMessage())
443+
.filter((key) -> key.startsWith(getExpiredKeyPrefix())).map((key) -> {
444+
int sessionIdBeginIndex = key.lastIndexOf(":") + 1;
445+
return key.substring(sessionIdBeginIndex);
446+
})
447+
.flatMap(this::deleteAndReturn)
446448
.map((session) -> {
447449
if (message.getChannel().equals(this.sessionDeletedChannel)) {
448450
return new SessionDeletedEvent(this, session);
@@ -451,6 +453,7 @@ private Mono<Void> onKeyDestroyedMessage(ReactiveSubscription.Message<String, St
451453
})
452454
.doOnNext(this::publishEvent)
453455
.then();
456+
// @formatter:on
454457
}
455458

456459
private void publishEvent(Object event) {

0 commit comments

Comments
 (0)