Skip to content

Add payload and headers for upload. - #4

Closed
lepture wants to merge 2 commits into
component:masterfrom
lepture:patch-1
Closed

Add payload and headers for upload.#4
lepture wants to merge 2 commits into
component:masterfrom
lepture:patch-1

Conversation

@lepture

Copy link
Copy Markdown
Contributor

When the server requires a CSRF token, we need to put it in the body or header.

@lepture

Copy link
Copy Markdown
ContributorAuthor

ping ...

@rauchg

Copy link
Copy Markdown

There should be a cleaner way of doing this, like returning a ref to the request object and then optionally adding headers and .end() is called in user-land.

@rauchg

Copy link
Copy Markdown

Actually it seems like you can do that right? By accessing upload.req ?

@lepture

Copy link
Copy Markdown
ContributorAuthor

@guille No, you can't do it with upload.req.

  1. payload is added to body
  2. headers should be set before send

@lepturelepture mentioned this pull request Aug 19, 2013
@lepture

Copy link
Copy Markdown
ContributorAuthor

@guille Another way to do this. #5

@lepture

Copy link
Copy Markdown
ContributorAuthor

in favor of #9

@lepturelepture closed this Jan 2, 2014
@lepture
lepture deleted the patch-1 branch January 2, 2014 08:45
Sign up for freeto 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.

2 participants

@lepture@rauchg