Sébastien Brochet

Results 7 comments of Sébastien Brochet

Thanks for your review! I added the changelog fragment and tried something for the integration tests

ok I'm progressing. I re-enabled the integration tests, like it was done in https://github.com/ansible-collections/community.aws/pull/763. I'm currently stuck with a permission error to create a new SSM document, so I opened...

hi @markuman, I already opened a MR for that in terminator: https://github.com/mattclay/aws-terminator/pull/192

Helle everyone, is there something additional I need to do to have this PR merged? Thanks!

Hi @spacewander, thanks a lot for your answer. I just tried with `rewrite_by_lua_block`, but I observe the same behavior. Interestingly, message in error logs already show the `X-Forwarded-For` ip already...

That's what I was afraid of :disappointed: Indeed, if I comment the check in https://github.com/nginx/nginx/blob/master/src/http/modules/ngx_http_realip_module.c#L152, the source IP is correctly changed. Any chance it's supported natively by openresty ? Anyway,...

It was a bit harder than expected, because the browser version is used in regexes to guess the browser engine, and, when truncated, it leads to different results. I extracted...