Spring/spring-framework
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #123 from violetagg/onsubscribe-state

Browse Source 
Make AbstractResponseBodySubscriber.onSubscribe thread-safe
This commit is contained in:
Arjen Poutsma
2016-06-29 09:51:30 +02:00
committed by GitHub
parent 11ed847aca 0605c0f3be
commit 92821d9300
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
spring-web-reactive/src/main/java/org/springframework/http/server/reactive/AbstractResponseBodySubscriber.java
View File

@@ -18,6 +18,7 @@ package org.springframework.http.server.reactive;
import java.io.IOException;
import java.nio.channels.Channel;
import java.util.Objects;
import java.util.concurrent.atomic.AtomicReference;
import javax.servlet.WriteListener;
@@ -169,11 +170,13 @@ abstract class AbstractResponseBodySubscriber implements Subscriber<DataBuffer>
@Override
void onSubscribe(AbstractResponseBodySubscriber subscriber,
Subscription subscription) {
if (BackpressureUtils.validate(subscriber.subscription, subscription)) {
Objects.requireNonNull(subscription, "Subscription cannot be null");
if (subscriber.changeState(this, REQUESTED)) {
subscriber.subscription = subscription;
if (subscriber.changeState(this, REQUESTED)) {
subscription.request(1);
}
subscription.request(1);
}
else {
super.onSubscribe(subscriber, subscription);
}
}
},