Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

Add line-delimited streaming support to crypto-client #908

Merged
merged 1 commit into from
May 3, 2019

Conversation

jgrund
Copy link
Member

@jgrund jgrund commented May 2, 2019

Fixes #907.

In order to process data as it comes in, it would be useful to have streaming support for GET requests.

Add streaming support for GET requests, and add a lib to emit data line by line.

Signed-off-by: Joe Grund jgrund@whamcloud.io

Fixes #907.

In order to process data as it comes in, it would be useful to have streaming support for GET requests.

Add streaming support for GET requests, and add a lib to emit data line by line.

Signed-off-by: Joe Grund <jgrund@whamcloud.io>
@jgrund jgrund requested review from johnsonw and utopiabound May 2, 2019 16:00
@jgrund jgrund self-assigned this May 2, 2019
@codecov-io
Copy link

codecov-io commented May 2, 2019

Codecov Report

Merging #908 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #908   +/-   ##
=======================================
  Coverage   95.42%   95.42%           
=======================================
  Files           2        2           
  Lines         153      153           
=======================================
  Hits          146      146           
  Misses          7        7

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc46595...30a4eb9. Read the comment docs.

@jgrund jgrund merged commit 7941b13 into master May 3, 2019
@jgrund jgrund deleted the line-streaming branch May 3, 2019 13:33
@jgrund jgrund added this to the Stratagem V1 milestone Jul 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add line-delimited streaming support to crypto-client
4 participants