Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Supports routing with multiple values for one header #1836

Closed
lianglli opened this issue Aug 4, 2023 · 1 comment · Fixed by #1864
Closed

Supports routing with multiple values for one header #1836

lianglli opened this issue Aug 4, 2023 · 1 comment · Fixed by #1864
Assignees
@lianglli
Labels
enhancement

Comments

@lianglli
Copy link
Member

lianglli commented Aug 4, 2023 •
edited
Loading

基于请求Header的高级路由基础上，可以指定具体匹配多个header值，在header和header值完全匹配的情况下，请求将被转发到指定的upstream。

Header值允许最多设置20个（默认），只要其中1个相匹配，请求就会被转发到指定的upstream。

无损动态生效，无需tengine reload。
@lianglli lianglli added this to the 3.0.1 milestone Aug 4, 2023
@lianglli lianglli self-assigned this Aug 4, 2023
@lianglli lianglli changed the title Supports canary routing with multiple values for one header Supports routing with multiple values for one header Aug 4, 2023
@lianglli lianglli removed this from the 3.0.1 milestone Sep 18, 2023
@lianglli lianglli linked a pull request Sep 18, 2023 that will close this issue
HTTP Route: supports routing based on multiple values, nginx var, or … #1864
Merged
@lianglli
Copy link
Member Author

lianglli commented Nov 1, 2023 

apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  annotations:
    nginx.ingress.kubernetes.io/canary: "true"
    nginx.ingress.kubernetes.io/canary-by-header: x-open
    nginx.ingress.kubernetes.io/canary-by-header-value: test1||test2
    nginx.ingress.kubernetes.io/ssl-redirect: "true"
  creationTimestamp: "2023-10-30T09:03:04Z"
  generation: 1
  name: tengine-ingress-hello-header-ing
  namespace: default
  resourceVersion: "28147345"
  uid: 7a5d1c7e-3da6-46f7-81a3-8182e1b420d1
spec:
  ingressClassName: opensource-ingress
  rules:
  - host: echo.w1.com
    http:
      paths:
      - backend:
          service:
            name: tengine-ingress-hello-service
            port:
              number: 80
        path: /
        pathType: Prefix
  tls:
  - hosts:
    - echo.w1.com
    secretName: https-server-1
status:
  loadBalancer:
    ingress:
    - {}

............................................................................

$ curl -i -k -H "x-open: test2" https://echo.w1.com
HTTP/2 200 
server: Tengine/3.1.0
date: Wed, 01 Nov 2023 08:43:46 GMT
content-type: text/html
content-length: 139
last-modified: Mon, 30 Oct 2023 08:15:02 GMT
strict-transport-security: max-age=0
ups-target-key: default-tengine-ingress-hello-service-80
x-protocol: HTTP/2.0
alt-svc: h3=":443"; ma=2592000,h3-29=":443"; ma=2592000

<html><head><title>HTTP Hello World</title></head><body><h1>Hello from tengine-ingress-hello-deployment-5f7699f65b-jt4kc</h1></body></html>

@lianglli lianglli closed this as completed Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lianglli lianglli

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

HTTP Route: supports routing based on multiple values, nginx var, or … lianglli/tengine
1 participant
@lianglli