Fix apicast replace_path to use remove #1285
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
It changes the routing policies injected into the proxy config for the backends of a product so they use the
replace_path
rule withremove_first
instead of withreplace
.Which issue(s) this PR fixes
Porta is currently generating proxy configs for products containing backends with paths with the following attribute in the corresponding injected routing policy:
This is causing apicast to route the incoming traffic, for example, from
<public-domain>/the-path/actual-backend-path
, to<private-domain>//actual-backend-path
. The correct backend endpoint would be<private-domain>/actual-backend-path
.We should use instead in the routing policy injected for the backend:
Closes THREESCALE-3593
Verification steps
routing
policies injected in the chain for the backends with path include areplace_path
rule in the following format:Example of part of a real proxy config generated using this PR
Special notes to the reviewer
Related to 3scale/APIcast#1122