lwip-users
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [lwip-users] Possible bug? Getting 200 Status code instead of 404, w


From: address@hidden
Subject: Re: [lwip-users] Possible bug? Getting 200 Status code instead of 404, when using LWIP_HTTPD_DYNAMIC_HEADER
Date: Tue, 21 Apr 2020 20:42:31 +0200
User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0

Am 21.04.2020 um 19:57 schrieb Marco Lazzaroni:
> tl;dr
> maybe in get_http_headers() the line "if (strstr(uri, "404.") == uri)"
> needs to be changed to "if (strstr(uri, "/404.") == uri)"

That might indeed be true...

>
> First of all thank you very much for the excellent work!
> Before filing a bug report I ask here because I'm in doubt.

Would you mind filing one so that this does not get forgotten?

Thanks,
Simon

>
> I get a HTTP 200 status code when doing a GET of a missing page. 
>
> LWIP version is last release got with git clone.
> I have prepared a 404.html page with this source code:
> <!DOCTYPE html>
> <html>
> <head>
> <title>404 Not Found</title>
> <meta name="viewport" content="width=device-width, initial-scale=1.0">
> <meta charset="utf8">
> </head>
> <body>
> <h1>404 Not Found</h1>
> </body>
> </html>
>
> In my lwipopts.h I have
> #define LWIP_HTTPD_DYNAMIC_HEADERS 1
> I generate the fsdata_custom.c file with
> makefsdata html -e -f:fsdata_custom.c
> (so the headers are not included twice)
>
> What I get when doing a GET of a missing page is wrong (it returns HTTP
> 200 status code):
>
> lavoro@saiga:~$ curl -i http://192.168.1.9/any_missing_page.html
> HTTP/1.0 200 OK
> Server: lwIP/2.2.0d (http://savannah.nongnu.org/projects/lwip)
> Content-Type: text/html
>
> <!DOCTYPE html>
> <html>
> <head>
> <title>404 Not Found</title>
> <meta name="viewport" content="width=device-width, initial-scale=1.0">
> <meta charset="utf8">
> </head>
>
> <body>
> <h1>404 Not Found</h1>
> </body>
> </html>
>
> If I set LWIP_HTTPD_DYNAMIC_HEADERS to 0 and generate fsdata_custom.c
> removing the -e switch (makefsdata html -e -f:fsdata_custom.c), the
> response is correct:
> lavoro@saiga:~$ curl -i http://192.168.1.9/any_missing_page.html
> HTTP/1.0 404 File not found
> Server: lwIP/1.3.1 (http://savannah.nongnu.org/projects/lwip)
> Content-type: text/html
>
> <!DOCTYPE html>
> <html>
> <head>
> <title>404 Not Found</title>
> <meta name="viewport" content="width=device-width, initial-scale=1.0">
> <meta charset="utf8">
> </head>
>
> <body>
> <h1>404 Not Found</h1>
> </body>
> </html>
>
> I suppose that the problem is that, with LWIP_HTTPD_DYNAMIC_HEADERS at
> 1, this piece of code in get_http_headers()
>
> if (strstr(uri, "404.") == uri) {
> hs->hdrs[HDR_STRINGS_IDX_HTTP_STATUS] =
> g_psHTTPHeaderStrings[HTTP_HDR_NOT_FOUND];
>
>
> ...does not behave as expected because while debugging I saw that uri
> equals "/404.html" and not "404.html", so the if condition is not true.
> I think that changing that line to 
>
> if (strstr(uri, "/404.") == uri) {
>
> (and the same for 400 an 501) would be enough; I've tested and it works:
>
> lavoro@saiga:~$ curl -i http://192.168.1.9/any_missing_page.html
> HTTP/1.0 404 File not found
> Server: lwIP/2.2.0d (http://savannah.nongnu.org/projects/lwip)
> Content-Type: text/html
>
> <!DOCTYPE html>
> <html>
> <head>
> <title>404 Not Found</title>
> <meta name="viewport" content="width=device-width, initial-scale=1.0">
> <meta charset="utf8">
> </head>
>
> <body>
> <h1>404 Not Found</h1>
> </body>
> </html>
>
> but I'm not sure whether is correct in all possible cases (I mean, all
> the possible #define combinations supported by LWIP).
>
> Could anyone patient enough to read till here give his opinion? Is my
> change correct? It's better to file a bug in order to get this corrected?
> Cheers
>  Marco
>
> _______________________________________________
> lwip-users mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/lwip-users
>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]