[FFmpeg-devel] [PATCH] avformat/http: print http headers at AV_LOG_DEBUG level

[Michael Niedermayer]

On Mon, Nov 04, 2013 at 06:37:52PM +0100, wm4 wrote:
> On Mon,  4 Nov 2013 16:05:15 +0100
> Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  libavformat/http.c |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/libavformat/http.c b/libavformat/http.c
> > index 7283477..4090ab0 100644
> > --- a/libavformat/http.c
> > +++ b/libavformat/http.c
> > @@ -333,7 +333,7 @@ static int process_line(URLContext *h, char *line, int line_count,
> >              p++;
> >          s->http_code = strtol(p, &end, 10);
> >  
> > -        av_dlog(NULL, "http_code=%d\n", s->http_code);
> > +        av_log(h, AV_LOG_DEBUG, "http_code=%d\n", s->http_code);
> >  
> >          /* error codes are 4xx and 5xx, but regard 401 as a success, so we
> >           * don't abort until all headers have been parsed. */
> > @@ -565,7 +565,7 @@ static int http_read_header(URLContext *h, int *new_location)
> >          if ((err = http_get_line(s, line, sizeof(line))) < 0)
> >              return err;
> >  
> > -        av_dlog(NULL, "header='%s'\n", line);
> > +        av_log(h, AV_LOG_DEBUG, "header='%s'\n", line);
> >  
> >          err = process_line(h, line, s->line_count, new_location);
> >          if (err < 0)
> 
> This should be very useful; I was about to make such a patch myself.
> Can you include printing the request sent to the server?

included, applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131105/ce91ad9b/attachment.asc>