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

fix setRequestHeaders for IE #40

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

Conversation

matiu
Copy link

@matiu matiu commented Dec 30, 2014

Fix setRequestHeaders for IE

@yemel
Copy link

yemel commented Dec 31, 2014

+1

1 similar comment
@pine
Copy link

pine commented Feb 7, 2015

👍

@Pomax
Copy link

Pomax commented Apr 23, 2015

@iriscouch can you please just merge this in and push a new version up? It's been a while, and this patch still fixes a critical bug in your code. Especially if this lib is no longer maintained, it'll be worth doing.

@PavelGavlik
Copy link

+1 This fixes a critial bug, which is present also in current Chrome.

@guillaume-duchemin
Copy link

+1 especially for >= IE 10

@DCKT
Copy link

DCKT commented Mar 28, 2016

Hi, no plan to merge this ? Thanks for your work :)

@Pomax
Copy link

Pomax commented Mar 28, 2016

Given that there has been no movement on this for over a year, we can safely assume this repo has been abandoned. Better alternatives (as of March 2016) are superagent (established as the clear winner between request, browser-request, etc) and node-fetch (an implementation of the Fetch API)

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.

7 participants