-
-
Notifications
You must be signed in to change notification settings - Fork 917
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
Add url
property to got.stream()
request and response objects
#229
Labels
Comments
The URLs that are included (in some redirects), also appear to be adding default ports in: // redirect.html -> (302) redirect2.html -> (301) redirected.html
require("got").stream("http://domain/redirect.html")
.on("redirect", response => console.log(response.url)); logs |
👍 We already do this for normal Interested in doing a pull request? :) |
url
property to got.stream()
request and response objects
kevva
added a commit
that referenced
this issue
Oct 19, 2016
floatdrop
pushed a commit
that referenced
this issue
Oct 20, 2016
* Add `requestUrl` to response in streaming mode (Fixes #229). * Set `requestUrl` inside `requestAsEventEmitter()`
floatdrop
pushed a commit
that referenced
this issue
Nov 1, 2016
* Add `requestUrl` to response in streaming mode (Fixes #229). * Set `requestUrl` inside `requestAsEventEmitter()`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Pretty well every other http request library provides a
url
property in these objects. Why doesn't this one?The text was updated successfully, but these errors were encountered: