Repository navigation
Conversation
…VCL (#87) For more details see https://ibexa.atlassian.net/browse/IBX-11181 and #87 Key changes: * Stripped client-supplied `X-Forwarded-*`, `Forwarded`, `X-Client-IP` and `Client-Cdn` headers in `vcl_recv` of `varnish5/6/7/9.vcl` unless the peer is in the new `trusted_proxies` ACL * Overwrote `X-Forwarded-For` with the real client IP for untrusted peers * Added the `trusted_proxies` ACL to `parameters.vcl` and documented it in the Varnish README * Added varnishtest coverage for trusted and untrusted peers, run in a new CI job * Added Behat coverage for header filtering and trusted proxy pass-through
For more details see https://ibexa.atlassian.net/browse/IBX-11181 and #111 Key changes: * Fixed the `ibexa/rest`, `ibexa/behat` and `ibexa/doctrine-schema` version constraints in `composer.json` to use `~4.6.x-dev`, aligning them with the other 4.6 Ibexa packages. * Removed the `ibexa/ci-scripts` dev dependency from `composer.json`. --------- Co-Authored-By: Andrew Longosz <alongosz@users.noreply.github.com>
# Conflicts: # composer.json
|
@alongosz : SonarCloud complains about duplications in |
No, this one can be skipped as pre-existing. There's no way to mark code duplication as resolved, so we will just merge this with red Sonar (mentioning this because it should be an exception, not a habit ;-) ). |
|
| "toflar/psr6-symfony-http-cache-store": "^4.2" | ||
| }, | ||
| "require-dev": { | ||
| "behat/behat": "^4.0@alpha", |
There was a problem hiding this comment.
It was in order to be able to do this here on 6.0 without changing minimum requierment
"ibexa/behat": "~6.0.x-dev",
see for instance admin-ui : ibexa/admin-ui@6cd6dfd
The commit where I added this here is named "Allowed the alpha behat packages ibexa/behat 6.0 requires", in an attempt to explain that
There was a problem hiding this comment.
Caution
This is a merge pull request
Related PRs: