From 2d3e68d912508c372599cff1252e35842eefbeae Mon Sep 17 00:00:00 2001 From: Jon Huhn Date: Tue, 22 Jun 2021 17:19:19 -0500 Subject: [PATCH] fix(demo): remove host headers in SMI policies The `:authority` header sent by the bookbuyer to the bookstore is actually `bookstore.bookstore:14001` where the bookstore's HTTPRouteGroup used in the manual demo specified a `host` header match of `bookstore.bookstore`, resulting in "response_code_details": "route_not_found" in the bookstore's logs for requests from the bookbuyer. This change removes the `host` header matches all of the demo manifests. Fixes #3616 Signed-off-by: Jon Huhn --- demo/deploy-traffic-specs.sh | 2 -- .../manifests/access/traffic-access-v1-allow-bookthief.yaml | 3 --- docs/example/manifests/access/traffic-access-v1.yaml | 3 --- 3 files changed, 8 deletions(-) diff --git a/demo/deploy-traffic-specs.sh b/demo/deploy-traffic-specs.sh index e13cf10178..ed213dfbc7 100755 --- a/demo/deploy-traffic-specs.sh +++ b/demo/deploy-traffic-specs.sh @@ -44,6 +44,4 @@ spec: - name: restock-books methods: - POST - headers: - - host: "bookwarehouse.$BOOKWAREHOUSE_NAMESPACE" EOF diff --git a/docs/example/manifests/access/traffic-access-v1-allow-bookthief.yaml b/docs/example/manifests/access/traffic-access-v1-allow-bookthief.yaml index 240a9fd96c..19453f4f4d 100644 --- a/docs/example/manifests/access/traffic-access-v1-allow-bookthief.yaml +++ b/docs/example/manifests/access/traffic-access-v1-allow-bookthief.yaml @@ -34,12 +34,9 @@ spec: methods: - GET headers: - - host: "bookstore.bookstore" - "user-agent": ".*-http-client/*.*" - "client-app": "bookbuyer" - name: buy-a-book pathRegex: ".*a-book.*new" methods: - GET - headers: - - host: "bookstore.bookstore" diff --git a/docs/example/manifests/access/traffic-access-v1.yaml b/docs/example/manifests/access/traffic-access-v1.yaml index 8349790533..500e75dbe9 100644 --- a/docs/example/manifests/access/traffic-access-v1.yaml +++ b/docs/example/manifests/access/traffic-access-v1.yaml @@ -34,12 +34,9 @@ spec: methods: - GET headers: - - host: "bookstore.bookstore" - "user-agent": ".*-http-client/*.*" - "client-app": "bookbuyer" - name: buy-a-book pathRegex: ".*a-book.*new" methods: - GET - headers: - - host: "bookstore.bookstore"