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

feat: added parsed headers to response object #71

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adampash
Copy link

@adampash adampash commented Nov 17, 2016

I don't know if this is something you're interested in adding, but I needed the response headers in the response, and this PR adds them. It also adds another dependency, which you might not want.

Anyway, I am using browser-request for a node/web project, and it's been great! I just needed this one adjustment to make it compatible with the response object from node request.

Thanks!

(Oh also, apologies for the semi-colon stuff. I can clean that up if you wanted to merge this.)

@jamesamcl
Copy link

Thanks for this. Shame the PR hasn't been merged but I'm using your fork for now.

@McPo
Copy link

McPo commented Jun 6, 2017

@iriscouch is this likely to be merged?

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.

3 participants