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

webapp: try not to use pycurl for live trace streaming #905

Closed
yugangw-msft opened this issue Sep 15, 2016 · 4 comments
Closed

webapp: try not to use pycurl for live trace streaming #905

yugangw-msft opened this issue Sep 15, 2016 · 4 comments

Comments

@yugangw-msft
Copy link
Contributor

yugangw-msft commented Sep 15, 2016

@haoc, for you get ramp up :-)
It would be great to use built-in http modules or the requests to accomplish it, so that we don't need to pull in the pycurl
The code is here

/CC:@derekbekoe

@yugangw-msft yugangw-msft added this to the Backlog milestone Sep 15, 2016
@derekbekoe
Copy link
Member

If we are going to use pycurl before our next release, we should add the dependency to our documentation (for the distributions that need it).

@yugangw-msft
Copy link
Contributor Author

For background, the reason to use pycurl is per kudu's document , which calls out using curl

@haocs
Copy link

haocs commented Sep 20, 2016

https://github.com/kennethreitz/requests/issues/3577 Some discussions of the issue that blocks replacing pycurl. Still trying to figure out how to let requests print data chunk by chunk as curl does.

@yugangw-msft yugangw-msft modified the milestones: Sprint 3, Backlog Sep 23, 2016
@yugangw-msft
Copy link
Contributor Author

Appears fixed. Thanks @haocs !

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

No branches or pull requests

3 participants