Robert Ma comments

Results 86 comments of


                                            Robert Ma

Populate HttpServletRequest.servletPath with pathPattern

@frojasg sorry I missed your reply. I was migrating some tracing/metrics code originally written for Jersey to our misk app. The code needs the path pattern as a tag, which...

Should servers be emitting many HTTP/2 control frames?

> Yes, this is normal behavior Is this a new behavior in 10.0.26 though? > Your application should read the data to avoid this By application do you mean the...

security: proxy-real-ip-cidr does NOT control use-forwarded-headers despite what the doc says

Sure, in a fresh minikube, install and configure ingress-nginx as follows: ``` kubectl apply -f https://raw.githubusercontent.com/kubernetes/ingress-nginx/main/deploy/static/provider/baremetal/deploy.yaml echo " apiVersion: networking.k8s.io/v1 kind: Ingress metadata: name: foo-bar annotations: kubernetes.io/ingress.class: nginx spec: rules:...

security: proxy-real-ip-cidr does NOT control use-forwarded-headers despite what the doc says

@longwuyuan I'm sorry. Could you point out which part of the reproduction steps are not clear? I didn't assume anything and reproduced the issue in a bare minikube using pretty...

security: proxy-real-ip-cidr does NOT control use-forwarded-headers despite what the doc says

Oh is it because of the `$POD_NAME` env var not found? That's actually a bug in your issue template: https://github.com/kubernetes/ingress-nginx/blob/739e85ce7cf2dbc13b710036a5c3fc465425c6e2/.github/ISSUE_TEMPLATE/bug_report.md?plain=1#L130-L131 the first line would actually produce a few pod names...

security: proxy-real-ip-cidr does NOT control use-forwarded-headers despite what the doc says

In case you missed the news, this project is being retired https://kubernetes.io/blog/2025/11/11/ingress-nginx-retirement/ --- Sent from my cellphone. On Mon, Dec 1, 2025, 21:14 Surya Keswani ***@***.***> wrote: > *surya-harvey* left...