change: responsed a better tip when failed to match any route. (apach…

…e#1013)

lua/apisix.lua

@@ -268,7 +268,7 @@ function _M.http_access_phase()
 
     local route = api_ctx.matched_route
     if not route then
-        return core.response.exit(404)
+        return core.response.exit(404, {error_msg = "failed to match any routes"})
     end
 
     if route.value.service_protocol == "grpc" then

lua/apisix/http/router/radixtree_host_uri.lua

@@ -158,7 +158,7 @@ function _M.match(api_ctx)
     end
 
     core.log.info("not find any matched route")
-    return core.response.exit(404)
+    return true
 end
 
 

lua/apisix/http/router/radixtree_uri.lua

@@ -98,7 +98,7 @@ function _M.match(api_ctx)
 
     if not uri_router then
         core.log.error("failed to fetch valid `uri` router: ")
-        return core.response.exit(404)
+        return true
     end
 
     core.table.clear(match_opts)
@@ -110,7 +110,7 @@ function _M.match(api_ctx)
     local ok = uri_router:dispatch(api_ctx.var.uri, match_opts, api_ctx)
     if not ok then
         core.log.info("not find any matched route")
-        return core.response.exit(404)
+        return true
     end
 
     return true

t/debug/hook.t

@@ -84,8 +84,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/global-rule.t

@@ -93,8 +93,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 
@@ -104,8 +104,8 @@ qr/404 Not Found/
 --- request
 GET /hello
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/hosts.t

@@ -64,8 +64,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 
@@ -77,8 +77,8 @@ GET /hello
 --- more_headers
 Host: not_found.com
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/invalid-route.t

@@ -56,8 +56,8 @@ qr/"value":"mexxxxxxxxxxxxxxx"/
 --- request
 GET /not_found
 --- error_code: 404
---- response_body_like eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- grep_error_log eval
 qr/\[error\].*/
 --- grep_error_log_out eval

t/node/invalid-service.t

@@ -57,8 +57,8 @@ qr/"value":"mexxxxxxxxxxxxxxx"/
 --- request
 GET /not_found
 --- error_code: 404
---- response_body_like eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- grep_error_log eval
 qr/\[error\].*/
 --- grep_error_log_out eval

t/node/invalid-upstream.t

@@ -56,8 +56,8 @@ qr/"value":"mexxxxxxxxxxxxxxx"/
 --- request
 GET /not_found
 --- error_code: 404
---- response_body_like eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- grep_error_log eval
 qr/\[error\].*/
 --- grep_error_log_out eval

t/node/merge-route.t

@@ -102,8 +102,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/not-exist-service.t

@@ -57,7 +57,7 @@ passed
 --- request
 GET /hello
 --- error_code: 404
---- response_body_like eval
+--- response_body eval
 qr/404 Not Found/
 --- wait_etcd_sync: 0.3
 --- grep_error_log eval

t/node/not-exist-upstream.t

@@ -67,8 +67,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body_like eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/remote-addr-ipv6.t

@@ -77,8 +77,8 @@ location /t {
 }
 --- request
 GET /t
---- response_body_like eval
-qr{.*404 Not Found.*}
+--- response_body eval
+qr/"error_msg":"failed to match any routes"/
 --- no_error_log
 [error]
 
@@ -89,8 +89,8 @@ qr{.*404 Not Found.*}
 --- request
 GET /not_found
 --- error_code: 404
---- response_body_like eval
-qr{.*404 Not Found.*}
+--- response_body eval
+qr/"error_msg":"failed to match any routes"/
 --- no_error_log
 [error]
 
@@ -129,7 +129,7 @@ close: 1 nil
 --- request
 GET /hello
 --- error_code: 404
---- response_body_like eval
-qr{.*404 Not Found.*}
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]

t/node/remote-addr.t

@@ -61,8 +61,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 
@@ -117,8 +117,8 @@ passed
 --- request
 GET /hello
 --- error_code: 404
---- response_body_like eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/remote_addrs.t

@@ -68,8 +68,8 @@ set_real_ip_from unix:;
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 
@@ -85,8 +85,6 @@ GET /hello
 --- more_headers
 Host: not_found.com
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
 --- no_error_log
 [error]
 

t/node/route-domain.t

@@ -64,8 +64,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/route-filter-func.t

@@ -64,8 +64,8 @@ passed
 --- request
 GET /hello?name=unknown
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/route-host.t

@@ -65,8 +65,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 
@@ -76,8 +76,8 @@ qr/404 Not Found/
 --- request
 GET /hello
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 
@@ -89,8 +89,8 @@ GET /hello
 --- more_headers
 Host: not_found.com
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/route-uris.t

@@ -64,8 +64,6 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
 --- no_error_log
 [error]
 

t/node/sanity-radixtree.t

@@ -64,8 +64,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 
@@ -149,7 +149,7 @@ qr/404 Not Found/
 --- request
 GET /hel
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]

t/node/upstream-domain.t

@@ -90,8 +90,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/upstream-ipv6.t

@@ -102,8 +102,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]
 

t/node/upstream.t

@@ -89,8 +89,8 @@ passed
 --- request
 GET /not_found
 --- error_code: 404
---- response_body eval
-qr/404 Not Found/
+--- response_body
+{"error_msg":"failed to match any routes"}
 --- no_error_log
 [error]