Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Actuator @ReadOperation on Flux cancels request after first element emitted #30095

Closed
ghenadiibatalski opened this issue Mar 8, 2022 · 1 comment
Assignees
Labels
type: bug A general bug
Milestone

Comments

@ghenadiibatalski
Copy link

Problem:
@ReadOperation on a custom actuator endpoint returning Flux cancels the request after first element, the same code wrapped as a @GetMapping of a @RestController returns full response as expected. The workaround is to return Mono but its a bit weird.

Spring Boot Version 2.6.4 created via initializr with Webflux and Actuator

demo.zip

Code:
Actuator Endpoint

@Endpoint(id = "demo")
@Component
public class DemoEndpoint {
    @ReadOperation
    public Flux<String> test() {
        return Flux.range(0, 10).map(String::valueOf).log();
    }
}

Demonstration Controller:

@RestController
public class DemoRestController {
    @GetMapping
    public Flux<String> test(){
        return Flux.range(0,10).map(String::valueOf).log();
    }
}

See sample project attached.

curl localhost:8080 -> 0123456789
curl localhost:8080/actuator/demo -> 0

Expected behavior: 2nd call delivers 0123456789

Log:

2022-03-07 14:35:55.185  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onSubscribe([Fuseable] FluxMapFuseable.MapFuseableSubscriber)
2022-03-07 14:35:55.186  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | request(1)
2022-03-07 14:35:55.187  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(0)
2022-03-07 14:35:55.201  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | request(127)
2022-03-07 14:35:55.201  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(1)
2022-03-07 14:35:55.201  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(2)
2022-03-07 14:35:55.202  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(3)
2022-03-07 14:35:55.202  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(4)
2022-03-07 14:35:55.202  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(5)
2022-03-07 14:35:55.202  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(6)
2022-03-07 14:35:55.203  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(7)
2022-03-07 14:35:55.203  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(8)
2022-03-07 14:35:55.203  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onNext(9)
2022-03-07 14:35:55.203  INFO 42052 --- [ctor-http-nio-2] reactor.Flux.MapFuseable.1               : | onComplete()
2022-03-07 14:36:45.169  INFO 42052 --- [ctor-http-nio-3] reactor.Flux.MapFuseable.2               : | onSubscribe([Fuseable] FluxMapFuseable.MapFuseableSubscriber)
2022-03-07 14:36:45.169  INFO 42052 --- [ctor-http-nio-3] reactor.Flux.MapFuseable.2               : | request(unbounded)
2022-03-07 14:36:45.169  INFO 42052 --- [ctor-http-nio-3] reactor.Flux.MapFuseable.2               : | onNext(0)
2022-03-07 14:36:45.169  INFO 42052 --- [ctor-http-nio-3] reactor.Flux.MapFuseable.2               : | cancel()
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 8, 2022
@wilkinsona
Copy link
Member

wilkinsona commented Mar 8, 2022

I spent a bit of time looking at this following the discussion on Gitter. Only a single element is returned due to this code:

Mono.from(Publisher) is documented as follows:

Expose the specified Publisher with the Mono API, and ensure it will emit 0 or 1 item. The source emitter will be cancelled on the first onNext.

There also appears to be problems with returning Flux when using Spring MVC and Jersey as well.

I can't remember if it was a conscious decision to only support Mono or if it's an oversight. If it was intentional, we should improve the documentation as we don't seem to say anything about supported return types at the moment. If it was an oversight, we'll have to decide if we want to support returning a Flux and, if we do, if we consider adding support to be a bug fix or an enhancement.

@wilkinsona wilkinsona added the for: team-meeting An issue we'd like to discuss as a team to make progress label Mar 8, 2022
@mhalbritter mhalbritter added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged for: team-meeting An issue we'd like to discuss as a team to make progress labels Mar 9, 2022
@mhalbritter mhalbritter added this to the 2.5.x milestone Mar 9, 2022
@wilkinsona wilkinsona self-assigned this Mar 10, 2022
@wilkinsona wilkinsona modified the milestones: 2.5.x, 2.5.11 Mar 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

No branches or pull requests

4 participants