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 openservicemesh#3616 Signed-off-by: Jon Huhn <[email protected]>
eduser25 · Jun 23, 2021 · 9d9d893 · 9d9d893
1 parent 50d4426
commit 9d9d893
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 8 deletions.
diff --git 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
@@ -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
@@ -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"