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.

Sign up for GitHub

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

Jump to bottom

Introduce @Cancellable annotation for Quarkus REST #45144

Merged
merged 1 commit into from
Dec 16, 2024
Merged

Introduce @Cancellable annotation for Quarkus REST #45144

merged 1 commit into from
Dec 16, 2024

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Dec 16, 2024

This annotation can be placed on REST methods
that return a single result async type
and allows the user to configure whether the
subscription should be cancelled if the
connection is closed before the result is available

@geoand geoand requested a review from cescoffier December 16, 2024 14:37
@geoand geoand mentioned this pull request Dec 16, 2024
Closed
@geoand
Introduce @Cancellable annotation for Quarkus REST
7bd308a 
This annotation can be placed on REST methods
that return a single result async type
and allows the user to configure whether the
subscription should be cancelled if the
connection is closed before the result is available

Closes: quarkusio#45141
@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Dec 16, 2024 •
edited by github-actions bot
Loading

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 7bd308a.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@geoand geoand merged commit ebe2306 into quarkusio:main Dec 16, 2024
46 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.18 - main milestone Dec 16, 2024
@geoand geoand deleted the #45141 branch December 17, 2024 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cescoffier cescoffier cescoffier approved these changes

Assignees
No one assigned
Labels
area/rest
Projects
None yet
Milestone
3.18 - main
Development

Successfully merging this pull request may close these issues.

Make cancellation handler for async requests configurable in quarkus-rest
2 participants
@geoand @cescoffier