-
Notifications
You must be signed in to change notification settings - Fork 42
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
Discussion about X-Requested-With header #21
Comments
I have another related issue: |
|
I know it's different, I said "related" :) We just faced it in comparison to Angular 2 Http service which not send that header so we hadn't that issues before, but now need to use nanoajax in one microlibrary. I will create issue and PR if have time later (if PRs merged here). |
Hello,
Is there particullar reason why the
X-Requested-With
header is set only is request has a body?(https://github.com/yanatan16/nanoajax/blob/master/index.js#L75)
Shouldn't the header be set for any non GET or HEAD request?
For example, if I'm making a DELETE request with no body, I think the header should be there... Currently, I have to add this header by hand.
I'm not very competent on that matter, so maybe I'm wrong... Please advise.
The text was updated successfully, but these errors were encountered: