Comments (3)
We can set ClientRequestContext
to CancellationHandler
in the following method.
CancellationHandler
invokes ClientRequestContext.cancel()
when it is set if CancellationListener.cancelled()
was called before.
By the way, this issue is labeled with sprint
so a sprint member is scheduled to work on this. 😅 How about looking for another issue?
from armeria.
Hi, @ikhoon ~
I understand that this issue requirement is To call RequestContext.cancel()
when gRPC level cancellation occurs.
Thank you for sharing your idea and implementation outline.
I'm not completely sure the following implementation and how this handler can access to ClientRequestContext
.
// CancellationHandler will defer the cancellation until `ClientRequestContext` to this.
final CancellationHandler handler = new CancellationHandler();
Do you have your idea about this already?
If so, it would be helpful if you could share.
from armeria.
Oh! Got it.
I'm sorry for misunderstanding of this labeling system.
Thank you for sharing your idea.
from armeria.
Related Issues (20)
- Test failure: `com.linecorp.armeria.server.Http1ServerDelayedCloseConnectionTest.shouldDelayDisconnectByServerSideIfClientDoesNotHandleConnectionClose()`
- Test failure: `com.linecorp.armeria.common.stream.InputStreamStreamMessageTest.thrownInputStream()` HOT 1
- Close a WebSocket when it is idle
- SslStoreProvider has been removed in spring boot 3.3.0, armeria webflux backward compatibility broken
- Consider setting minimum JDK version to 11 or 17 HOT 1
- Healthcheck service returning unhealthy between tests if service is reconfigured
- Suggest adding an example when armeria + spring is used for docs.
- Support for degraded health HOT 1
- `RetryingClient` gets deadlocked when a decorator returns a failed response.
- Let `GrpcService` specify a maximum bound for `grpc-timeout` HOT 6
- Consider dropping `:` from path pattern
- Set `RequestLog#content` for `AnnotatedService` if a pojo is requested/returned
- Provide an easy way for users to leave `ClientConnectionTimings` to logs
- Include Current Attempt Number in Logs When Retrying
- Apply circuit breaker per connection
- Add ServerBuilder.http1ConnectionCloseDelayMillis() HOT 1
- Test failure: `com.linecorp.armeria.server.Http1ServerDelayedCloseConnectionTest.shouldWaitForDisconnectByClientSideFirst()`
- Test failure: `com.linecorp.armeria.internal.client.dns.DefaultDnsResolverTest.[2] resolvedAddressType=IPV6_PREFERRED`
- RoutingContext.withPath() doesn't respect Flags.allowSemicolonInPathComponent() HOT 1
- Test failure: `com.linecorp.armeria.testing.junit5.server.ServerExtensionTest.requestContextCaptor()` HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from armeria.