Patrick Taibel

Results 8 comments of Patrick Taibel

[TT-6202] Golang Virtual Endpoints should not produce runtime error in log message although request and response is normal

We did run into the same issue last week. @mhuaco Additionally to the fix from @uddmorningsun for the `successHandler` there is another issue not present in master but on the...

Fix invalid memory address or nil pointer dereference in Golang Virtual Endpoints

I'd suggest introducing a new local variable `successHandler` as already done with `logger` and `handler`. Else you modify the shared `GoPluginMiddleware` on every call there.

Fix setting global_size_limit not enabling RequestSizeLimitMiddleware

I've added a small test for this. Thank you very much for the pretty much finished test snippet!

Fix setting global_size_limit not enabling RequestSizeLimitMiddleware

I've rebased the branch onto the current master. The previous test run seems to have failed on test `--- FAIL: TestEnforcedTimeout (4.27s)`. As far as I can tell, this has...

[TT-5186] Add a global request body size limit

I've added a small test for this. Thank you very much for the pretty much finished test snippet! I've changed the way to set the config though. With your way...

[TT-5186] Add a global request body size limit

@titpetric Those two PRs are independent of each other (and both branches are directly based on master). This one adds a global serverwide hard limit on the request body size....

Fix transform response context containing wrong rewrite path

I've adjusted the existing tests for the HeaderInjector response middleware to include this kind of url rewriting too. Additionally, I've added a check for the urlRewritePath context in the `TestRewriter`...

[TT-3158] Fix for quantity match expressions on method transform paths

I would like to revive this pull request. Our issue here is that every field that is parsed as `URLSpec` does not support regex quantifiers that are wrapped with `{...