Re: [PATCH 1/2] imap-send.c: implements the GIT_CURL_DEBUG environment variable
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Apr 1, 2016, 20:25 UTC
- Message-ID
- <CAPig+cRrmdiz4fHAX3xg6CinSOPBv207RZLaacRBkoDNHLaPwA@mail.gmail.com>
- In-Reply-To
- <1459507482-36678-1-git-send-email-gitter.spiros@gmail.com>
On Fri, Apr 1, 2016 at 6:44 AM, Elia Pinto <gitter.spiros@gmail.com> wrote:
Show 5 quoted lines
> Implements the GIT_CURL_DEBUG environment variable to allow a greater > degree of detail of GIT_CURL_VERBOSE, in particular the complete > transport header and all the data payload exchanged. > It might be useful if a particular situation could require a more > thorough debugging analysis.
In addition to review comments by others, why are the new curl_dump() and curl_trace() functions duplicated in both patches rather than factored out to a shared implementation?
Show 96 quoted lines
> Signed-off-by: Elia Pinto <gitter.spiros@gmail.com>
> ---
> diff --git a/imap-send.c b/imap-send.c
> @@ -1395,6 +1395,96 @@ static int append_msgs_to_imap(struct imap_server_conf *server,
> }
>
> #ifdef USE_CURL_FOR_IMAP_SEND
> +
> +static
> +void curl_dump(const char *text,
> + FILE * stream, unsigned char *ptr, size_t size, char nohex)
> +{
> + size_t i;
> + size_t c;
> +
> + unsigned int width = 0x10;
> +
> + if (nohex)
> + /* without the hex output, we can fit more on screen */
> + width = 0x40;
> +
> + fprintf(stream, "%s, %10.10ld bytes (0x%8.8lx)\n",
> + text, (long)size, (long)size);
> +
> + for (i = 0; i < size; i += width) {
> +
> + fprintf(stream, "%4.4lx: ", (long)i);
> +
> + if (!nohex) {
> + /* hex not disabled, show it */
> + for (c = 0; c < width; c++)
> + if (i + c < size)
> + fprintf(stream, "%02x ", ptr[i + c]);
> + else
> + fputs(" ", stream);
> + }
> +
> + for (c = 0; (c < width) && (i + c < size); c++) {
> + /* check for 0D0A; if found, skip past and start a new line of output */
> + if (nohex && (i + c + 1 < size) && ptr[i + c] == 0x0D
> + && ptr[i + c + 1] == 0x0A) {
> + i += (c + 2 - width);
> + break;
> + }
> + fprintf(stream, "%c",
> + (ptr[i + c] >= 0x20)
> + && (ptr[i + c] < 0x80) ? ptr[i + c] : '.');
> + /* check again for 0D0A, to avoid an extra \n if it's at width */
> + if (nohex && (i + c + 2 < size)
> + && ptr[i + c + 1] == 0x0D
> + && ptr[i + c + 2] == 0x0A) {
> + i += (c + 3 - width);
> + break;
> + }
> + }
> + fputc('\n', stream); /* newline */
> + }
> + fflush(stream);
> +}
> +
> +static
> +int curl_trace(CURL * handle, curl_infotype type,
> + char *data, size_t size, void *userp)
> +{
> + const char *text;
> + (void)handle; /* prevent compiler warning */
> +
> + switch (type) {
> + case CURLINFO_TEXT:
> + fprintf(stderr, "== Info: %s", data);
> + default: /* in case a new one is introduced to shock us */
> + return 0;
> +
> + case CURLINFO_HEADER_OUT:
> + text = "=> Send header";
> + break;
> + case CURLINFO_DATA_OUT:
> + text = "=> Send data";
> + break;
> + case CURLINFO_SSL_DATA_OUT:
> + text = "=> Send SSL data";
> + break;
> + case CURLINFO_HEADER_IN:
> + text = "<= Recv header";
> + break;
> + case CURLINFO_DATA_IN:
> + text = "<= Recv data";
> + break;
> + case CURLINFO_SSL_DATA_IN:
> + text = "<= Recv SSL data";
> + break;
> + }
> +
> + curl_dump(text, stderr, (unsigned char *)data, size, 1);
> + return 0;
> +}