Skip to content

fix: loosing request header (#4858)#4871

Merged
jasonsaayman merged 2 commits intoaxios:v0.xfrom
WillianAgostini:fix-loosing-request-headers
Sep 26, 2022
Merged

fix: loosing request header (#4858)#4871
jasonsaayman merged 2 commits intoaxios:v0.xfrom
WillianAgostini:fix-loosing-request-headers

Conversation

@WillianAgostini
Copy link
Copy Markdown
Contributor

@WillianAgostini WillianAgostini commented Jul 21, 2022 •
edited
Loading

Fixes #4858

As described by #4858 there are problems that was loosing request headers.

Fix the error when pass headers parameters from class.

class Header {};
let myHeaders = new Header();
myHeaders["X-Foo"] = "bar";
myHeaders["X-Test"] = "test";
myHeaders["X-Time"] = Date.now();
  
axios.post('http://cors-test.appspot.com/test', { name: 'foo' }, { headers: myHeaders })
.then((res) => {
       console.log('res---',res.config.headers);
 })

@WillianAgostini WillianAgostini mentioned this pull request Jul 21, 2022
Closed
@jasonsaayman jasonsaayman merged commit 80b546c into axios:v0.x Sep 26, 2022
@baby636 baby636 mentioned this pull request Oct 17, 2022
Open
@gmasclet gmasclet mentioned this pull request Jul 8, 2024
Open
This was referenced Oct 8, 2025
Open
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasonsaayman jasonsaayman jasonsaayman approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@WillianAgostini @jasonsaayman