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

Implemented multipart/form-data POSTS with only parameters and without files. #385

Merged
merged 1 commit into from
Sep 4, 2013

Conversation

meixger
Copy link
Contributor

@meixger meixger commented Jun 3, 2013

Implemented multipart/form-data POSTS with only parameters and without
files.

Implemented multipart/form-data POSTS with only parameters and without
files.
@@ -293,7 +298,7 @@ private string EncodeParameters()

private void PreparePostBody(HttpWebRequest webRequest)
{
if(HasFiles)
if (HasFiles || AlwaysMultipartFormData)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation is wrong here.

@haacked
Copy link
Contributor

haacked commented Sep 4, 2013

Thanks for the PR! The code looks good. Just one style issue. Could you tell me what scenarios this is used for just so I have more context for the change?

@meixger
Copy link
Contributor Author

meixger commented Sep 4, 2013

It's required for AlpineBits. "[..] is a definition of an interface, based on OTA and fitted to the Alpine Tourism, for the exchange of touristic data".

An [AlpineBits XML] message must be transfered via a POST following the multipart/form-data encoding scheme.

So i have 1 or more parameters but no files.

PS: sorry for the style issue – had disabled whitespace on my diff tool

haacked added a commit that referenced this pull request Sep 4, 2013
Implemented multipart/form-data POSTS with only parameters and without files.
@haacked haacked merged commit 60aa64f into restsharp:master Sep 4, 2013
@haacked
Copy link
Contributor

haacked commented Sep 4, 2013

Thanks!

thumbsup-goggles

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.

None yet

2 participants