cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] merge user-defined content-type with content-type built by formdata.c

From: <yves.lejeune_at_kodak.com>
Date: Wed, 26 Jul 2006 16:06:18 +0200

(i hope this will be attached to the right thread, i only subscribed to
the daily digest).

> > Here is a patch of http.c for curl 7.15.4, which make it possible to
> > specify a Content-Type header for the HTTP message,
>
> Gee, I was almost sure this was already possible! :-O Thanks for taking
on the
> task of fixing this.
>
> > I hope you will like the patch, and insert it in the code base: please
tell
> > me if there is something wrong.
>
> I have two requests/suggestions on this patch:
>
> 1 - please attach the patch as it line wraps heavily and that makes it
hard
> to apply
>
> 2 - I would prefer if you instead of the "weird magic" trying to replace
the
> content-type string, made the Curl_getFormData() take a new
argument that
> would either be NULL for default type or a custome string defining
what
> content-type to use.

OK, this makes sense - no more need to have code in http.c which relies on
formdata.c internals.

Here is the new version of the patch.

Best regards,
Yves Lejeune.

Received on 2006-07-26