Daniel

Results 26 comments of Daniel

We could also leverage the following libraries: https://pkg.go.dev/github.com/s-kozlov/goback/regexp/syntax https://pkg.go.dev/github.com/dlclark/regexp2#section-readme

We should implement https://github.com/nginxinc/kubernetes-ingress/issues/5388 before fixing this issue.

@angeloxx The idea is to have the schema embedded in the helm chart. We will look into in the due course.

We have a PR merged to fix this issue https://github.com/nginxinc/kubernetes-ingress/pull/5493 We need to validate the fix.

We have tests that currently pass with this scenario, we need to investigate them.

Hi @llomgui, We reviewed your issue and the team recommendation is to get the contents of your PR and port it into the [Open ID connect reference implementation](https://github.com/nginxinc/nginx-openid-connect)

@llomgui thank you. Once that PR is merged in the nginx-openid-connect, files should get copied over to https://github.com/nginxinc/kubernetes-ingress/tree/main/internal/configs/oidc via a PR. You can close this PR for now.

@llomgui you are right, we can leave this PR open. Once we get the nginx-openid-connect PR in we can update here.

Reopened as it is one of the stories to complete https://github.com/nginxinc/kubernetes-ingress/issues/4404

Hi @llomgui, we are closing this PR and our team is taking over the https://github.com/nginxinc/kubernetes-ingress/issues/4989 implementation. You can follow the progress in the issue, we are working on it in...