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.

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

Add support for http header routing #1222

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

erich23
Copy link

@erich23 erich23 commented Dec 17, 2024

tableMemory.store now holds a list of *httpv1alpha1.HTTPScaledObject

When routing, iterate through list of *httpv1alpha1.HTTPScaledObject and match the headers based on passed in http headers. Here are the rules:

  1. After matching on host and pathPrefix, if any http request header and matches any headers in HTTPScaledObject.spec.Headers then route to the first match.
  2. If no http request header matches any headers in in any HTTPScaledObject, then route to one which does not specify HTTPScaledObject.spec.Headers
  3. If there is no suchHTTPScaledObject which specifies no .spec.Headers, fail routing.

Checklist

Fixes #

@erich23 erich23 requested a review from a team as a code owner December 17, 2024 19:43
@erich23 erich23 marked this pull request as draft December 18, 2024 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant