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

Mutiny Dropped Exception on SSE Cancel #27489

Closed
jimbogithub opened this issue Aug 24, 2022 · 4 comments · Fixed by #27559
Closed

Mutiny Dropped Exception on SSE Cancel #27489

jimbogithub opened this issue Aug 24, 2022 · 4 comments · Fixed by #27559
Assignees
Labels
area/rest-client kind/bug Something isn't working
Milestone

Comments

@jimbogithub
Copy link

Describe the bug

Undesirable ERROR message appears in logs, e.g:

2022-08-25 11:30:51,013 ERROR [io.qua.mut.run.MutinyInfrastructure] (vert.x-eventloop-thread-2) Mutiny had to drop the following exception: io.vertx.core.http.HttpClosedException: Connection was closed

Expected behavior

Would prefer not to have these appearing in Production logs. Can they be avoided or suppressed?

Actual behavior

No response

How to Reproduce?

https://github.com/jimbogithub/mutiny-sse

Run the server and the client. Observe client logs.

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

Mutiny SSE otherwise looks much better than legacy SSE (#23997).

@jimbogithub jimbogithub added the kind/bug Something isn't working label Aug 24, 2022
@quarkus-bot
Copy link

quarkus-bot bot commented Aug 24, 2022

/cc @cescoffier, @jponge

@geoand
Copy link
Contributor

geoand commented Aug 25, 2022

When do these show up?

@jimbogithub
Copy link
Author

When do these show up?

Does the reproducer not work for you?

@geoand
Copy link
Contributor

geoand commented Aug 26, 2022

I missed that, sorry

@cescoffier cescoffier self-assigned this Aug 29, 2022
cescoffier added a commit to cescoffier/quarkus that referenced this issue Aug 29, 2022
cescoffier added a commit to cescoffier/quarkus that referenced this issue Aug 29, 2022
@quarkus-bot quarkus-bot bot added this to the 2.13 - main milestone Aug 29, 2022
fercomunello pushed a commit to fercomunello/quarkus that referenced this issue Aug 31, 2022
@gsmet gsmet modified the milestones: 2.13 - main, 2.12.1.Final Sep 5, 2022
gsmet pushed a commit to gsmet/quarkus that referenced this issue Sep 5, 2022
evanchooly pushed a commit to nenros/quarkus that referenced this issue Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/rest-client kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants