Datong Sun

Results 69 comments of Datong Sun

Hello @fdeschamps , Thanks for your contribution. I have one curiosity, since the `/nginx_status` location is marked as `internal`, I don't think it will be possible to fetch it using...

@mayocream Could you add the changelog entry for this PR? I think it will be forgotten and not done if merged now.

@kikito This is less concerning to me, even in the rare case user relied on query parameters being ignored, having unneeded query parameters should not affect anything anyway as long...

@hbagdi This got on our radar already, and is high priority to fix.

This has been addressed in https://github.com/Kong/kong/pull/9451, and will be released in the upcoming 3.0.1 release.

Hello @isavcic, Thanks for your contribution! Could you change the test file https://github.com/Kong/kong/blob/master/spec/03-plugins/27-aws-lambda/03-iam-ec2-credentials_spec.lua as well so this new format is tested?

@bungle Could you resolve the conflicts on this PR?

Could you test with the latest Kong release and see if the issue still exists? Kong 2.2.1 is a really old release. What is your config size? How many routes/services/upstreams...

We recently got affected by this change as well. IMO we should not add too much restrictions to the `ngx.req.read_body()` API for HTTP/2 as it severely limits the usefulness of...