Browse Source

Fix race condition in WriteResultPublisher

between subscription and error from the Publish

Closes gh-25096
pull/25758/head
Rossen Stoyanchev 5 years ago
parent
commit
c0c3c01afd
  1. 8
      spring-web/src/main/java/org/springframework/http/server/reactive/WriteResultPublisher.java

8
spring-web/src/main/java/org/springframework/http/server/reactive/WriteResultPublisher.java

@ -1,5 +1,5 @@ @@ -1,5 +1,5 @@
/*
* Copyright 2002-2018 the original author or authors.
* Copyright 2002-2020 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@ -182,6 +182,9 @@ class WriteResultPublisher implements Publisher<Void> { @@ -182,6 +182,9 @@ class WriteResultPublisher implements Publisher<Void> {
@Override
void publishError(WriteResultPublisher publisher, Throwable ex) {
publisher.errorBeforeSubscribed = ex;
if(State.SUBSCRIBED.equals(publisher.state.get())) {
publisher.state.get().publishError(publisher, ex);
}
}
},
@ -200,6 +203,9 @@ class WriteResultPublisher implements Publisher<Void> { @@ -200,6 +203,9 @@ class WriteResultPublisher implements Publisher<Void> {
@Override
void publishError(WriteResultPublisher publisher, Throwable ex) {
publisher.errorBeforeSubscribed = ex;
if(State.SUBSCRIBED.equals(publisher.state.get())) {
publisher.state.get().publishError(publisher, ex);
}
}
},

Loading…
Cancel
Save