http: Print an error message for Authorization Required, too
authorMartin Storsjö <martin@martin.st>
Thu, 10 Nov 2011 16:21:17 +0000 (18:21 +0200)
committerMartin Storsjö <martin@martin.st>
Fri, 11 Nov 2011 12:07:48 +0000 (14:07 +0200)
The error was hidden before, to avoid showing an error on the
first request where no auth has been provided, when the server
indicates which authentication method to use.

Now the error is printed if an authentication method was used,
but failed.

Signed-off-by: Martin Storsjö <martin@martin.st>
libavformat/http.c

index 83ffc0b..093bb82 100644 (file)
@@ -236,7 +236,8 @@ static int process_line(URLContext *h, char *line, int line_count,
 
         /* error codes are 4xx and 5xx, but regard 401 as a success, so we
          * don't abort until all headers have been parsed. */
-        if (s->http_code >= 400 && s->http_code < 600 && s->http_code != 401) {
+        if (s->http_code >= 400 && s->http_code < 600 && (s->http_code != 401
+            || s->auth_state.auth_type != HTTP_AUTH_NONE)) {
             end += strspn(end, SPACE_CHARS);
             av_log(h, AV_LOG_WARNING, "HTTP error %d %s\n",
                    s->http_code, end);