Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Update http.js #4101

Closed
wants to merge 1 commit into from
Closed

Update http.js #4101

wants to merge 1 commit into from

Conversation

MrRaindrop
Copy link
Contributor

.get['My-header'] = 'value' will never work when .get is undefined.

.get['My-header'] = 'value' will never work when .get is undefined.
@mary-poppins
Copy link

Thanks for the PR!

  • Contributor signed CLA now or in the past
    • If you just signed, leave a comment here with your real name
  • PR's commit messages follow the commit message format

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@ghost ghost assigned petebacondarwin Sep 22, 2013
petebacondarwin pushed a commit that referenced this pull request Sep 25, 2013
Initially, `$httpProvider.defaults.headers.get` is `undefined`, so
`$httpProvider.defaults.headers.get['My-Header']='value'` will throw an
error.

Closes #4101
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
Initially, `$httpProvider.defaults.headers.get` is `undefined`, so
`$httpProvider.defaults.headers.get['My-Header']='value'` will throw an
error.

Closes angular#4101
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
Initially, `$httpProvider.defaults.headers.get` is `undefined`, so
`$httpProvider.defaults.headers.get['My-Header']='value'` will throw an
error.

Closes angular#4101
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants