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

Expose ClientCall in ClientReaderHandler and ClientRWHandler #87

Merged
merged 1 commit into from
Aug 22, 2019

Conversation

cocreature
Copy link
Collaborator

This allows you to cancel the call from within the callback using
clientCallCancel.

This allows you to cancel the call from within the callback using
`clientCallCancel`.
@Gabriella439
Copy link
Contributor

@cocreature: Thank you!

Just FYI, we may refactor {Client,Server}Request and {Client,Server}Result to be symmetric in the near future, although that is out of scope for this pull request.

The reason I mention it is that at some point the type of ClientReaderRequest/ServerWriterRequest might be unified to both be something like:

data RequestStatus = Cancel | Continue

ReaderRequest
    :: MetadataMap -> TimeoutSeconds -> (response -> IO RequestStatus) -> ReaderRequest

... and I wanted to let you know so that such a change doesn't take you by surprise later on. We'll cc: you on those pull requests so that you can review them.

@Gabriella439 Gabriella439 merged commit a26497c into awakesecurity:master Aug 22, 2019
@cocreature
Copy link
Collaborator Author

@Gabriel439 Thanks for the heads-up! Sounds like a very reasonable change.

RichardWarfield pushed a commit to litxio/gRPC-haskell that referenced this pull request Apr 25, 2023
RichardWarfield pushed a commit to litxio/gRPC-haskell that referenced this pull request Apr 25, 2023
These were the changes I had to made for Hackage to
accept the package upload
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants