Skip to content

Commit

Permalink
Fix issues with recent Emissary-Ingress version (#211)
Browse files Browse the repository at this point in the history
* Added mitigation of the matching problem due the port in the GRPC Host header.

Signed-off-by: Lukas Grundmann <[email protected]>

* Emissary Ingress subchart deploys default listener by default.

Signed-off-by: Lukas Grundmann <[email protected]>
  • Loading branch information
lukasgr90 authored Jul 28, 2022
1 parent 69d5ebe commit b0f6c37
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions build/package/helm/monoskope/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,20 @@ ambassador:
enabled: false
adminService:
create: false
createDefaultListeners: true
module:
# from https://github.com/emissary-ingress/emissary/issues/2276
# strips away any port from the Host header before route mapping
lua_scripts: |
function envoy_on_request(request_handle)
local authority = request_handle:headers():get(":authority")
if(string.find(authority, ":") ~= nil)
then
local authority_index = string.find(authority, ":")
local stripped_authority = string.sub(authority, 1, authority_index - 1)
request_handle:headers():replace(":authority", stripped_authority)
end
end
strip_matching_host_port: true # necessary for gRPC, see https://www.getambassador.io/docs/emissary/latest/howtos/grpc/#mappings-with-hosts

scimserver:
Expand Down

0 comments on commit b0f6c37

Please sign in to comment.