Merge branch 'fix/incorrect_response_status_code_captive_portal' into 'master'

Ashish285 · Ashish285 · commit b768059299fc · 2025-11-04T17:11:59.000+08:00
fix(http_server): Fixed response status code for temporary redirect

Closes IDFGH-16702

See merge request espressif/esp-idf!43061
diff --git a/examples/protocols/http_server/captive_portal/main/main.c b/examples/protocols/http_server/captive_portal/main/main.c
@@ -127,7 +127,7 @@ static const httpd_uri_t root = {
 esp_err_t http_404_error_handler(httpd_req_t *req, httpd_err_code_t err)
 {
     // Set status
-    httpd_resp_set_status(req, "302 Temporary Redirect");
+    httpd_resp_set_status(req, "303 See Other");
     // Redirect to the "/" root directory
     httpd_resp_set_hdr(req, "Location", "/");
     // iOS requires content in the response to detect a captive portal, simply redirecting is not sufficient.

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ static const httpd_uri_t root = {`
`127`	`127`	`esp_err_t http_404_error_handler(httpd_req_t *req, httpd_err_code_t err)`
`128`	`128`	`{`
`129`	`129`	`// Set status`
`130`		`- httpd_resp_set_status(req, "302 Temporary Redirect");`
	`130`	`+ httpd_resp_set_status(req, "303 See Other");`
`131`	`131`	`// Redirect to the "/" root directory`
`132`	`132`	`httpd_resp_set_hdr(req, "Location", "/");`
`133`	`133`	`// iOS requires content in the response to detect a captive portal, simply redirecting is not sufficient.`