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

Close the response body #222

Merged
1 commit merged into from
Jul 25, 2014
Merged

Close the response body #222

1 commit merged into from
Jul 25, 2014

Conversation

fraenkel
Copy link

The response body needs to be closed to properly close the client connection.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/75678010.

@lcddave
Copy link
Contributor

lcddave commented Jul 25, 2014

I heard in retro today you made quite a few discoveries. Any more coming down the pipeline? I can still put them up top for a few more hours.

@fraenkel
Copy link
Author

There are a couple more, but I have to track them down first.

@lcddave lcddave added the bug label Jul 25, 2014
@ghost
Copy link

ghost commented Jul 25, 2014

Everything checks out when running locally. Will pull this in.

ghost pushed a commit that referenced this pull request Jul 25, 2014
@ghost ghost merged commit 8e0ff1b into cloudfoundry:master Jul 25, 2014
haydonryan pushed a commit to haydonryan/cli that referenced this pull request Nov 7, 2014
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants