solo-io / gloo

The Cloud-Native API Gateway and AI Gateway
https://docs.solo.io/
Apache License 2.0
4.09k stars 446 forks source link

Support regex and headers matchers for delegateActions #8093

Open bdecoste opened 1 year ago

bdecoste commented 1 year ago

Version

1.13.x (latest stable)

Is your feature request related to a problem? Please describe.

Gloo requires a prefix matcher on a VS route delegateAction which limits our ability to delegate routes to RTs. We would like to be able to delegate to RTs using regex or headers matchers. If you configure a delegateAction with a regex, for example, you get this:

invalid route: routes with delegate actions must use a prefix matcher

    routes:
    - delegateAction:
        name: regex
        namespace: gloo-system
      matchers:
      - regex: /foo/(bar/|(xyz/1[.]0))

Describe the solution you'd like

Support all matcher types for delegateAction

Describe alternatives you've considered

Don't delegate non-prefix matchers to RTs

Additional Context

No response

soloio-bot commented 1 year ago

Zendesk ticket #1807 has been linked to this issue.

kdorosh commented 1 year ago

is it too constrained if the parent cannot use regex? or if the parent regex must match identically?

github-actions[bot] commented 4 months ago

This issue has been marked as stale because of no activity in the last 180 days. It will be closed in the next 180 days unless it is tagged "no stalebot" or other activity occurs.