Nick Young
Nick Young
Sounds like "change things to officially support lowercase strings" is probably more consistent with prior art, I guess. I'd like to see some more feedback from others before we go...
Still an issue we need to fix. /lifecycle frozen
Yes, thanks for logging this one to remind us @johnbellessa. Please feel free to bump it if we don't get to it. 😄
/remove-lifecycle stale
@saintdle Is this one still ongoing? Is there anything else you need here?
tbh I think this needs a @squeed review, passing to him.
Tagging in @squeed to take over the sig-k8s review, as I'm not very familiar with the CNI code.
Okay, there are a _lot_ of issues being reported here, so I'm going to go through this one-by-one. @julian-perge, firstly, the opening issue. Cilium only has access to TLS certificates...
@xadhatter, it looks like you're running into a property of the current Cilium implementation's filter chain setup - that there is a single filter chain for all matches _by protocol_,...
As @syedazeez337 has pointed out in #39480, the way to fix this is to have Cilium's request handling add special header check exceptions for when a redirect is added, to...