Skip to content

Conversation

@evankanderson
Copy link
Member

Summary

With the previous definition, either as:

rules:
- matches: []

or

rules:
- backendRefs: ...
  # no matches

I would get an error:

Details: unexpected HTTP status code received from server: 404 (Not Found); transport: received unexpected content-type "application/json"

This configuration seems to successfully route traffic (proven with the following config):

grpc_server:
  host: api.custcodian.dev
  port: 443
  insecure: false
identity:
  cli:
    issuer_url: https://auth.custcodian.dev
    client_id: minder-cli
    realm: login

Change Type

Mark the type of change your PR introduces:

  • Bug fix (resolves an issue without affecting existing features)
  • Feature (adds new functionality without breaking changes)
  • Breaking change (may impact existing functionalities or require documentation updates)
  • Documentation (updates or additions to documentation)
  • Refactoring or test improvements (no bug fixes or new functionality)

Testing

Manual testing on custcodian.dev domain.

Review Checklist:

  • Reviewed my own code for quality and clarity.
  • Added comments to complex or tricky code sections.
  • Updated any affected documentation.
  • Included tests that validate the fix or feature.
  • Checked that related changes are merged.

@evankanderson evankanderson requested a review from a team as a code owner March 10, 2025 22:56
@coveralls
Copy link

Coverage Status

coverage: 57.272%. remained the same
when pulling 6f312cb on evankanderson:http-routes-fix
into a8d2baa on mindersec:main.

@evankanderson evankanderson merged commit 5b9a701 into mindersec:main Mar 11, 2025
26 checks passed
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.

3 participants