Do not rewrite outbound URL if the source domain is the active domain

The merge requests updates the Domain Source OutboundPathProcessor to not rewrite the outbound URL if the source domain is the active domain

Hi @idebr,

I'm stuck on a functional test that runs fine on current version (D11) but fails on previous major (D10) when executed via a Gitlab CI pipeline.

Test works perfectly fine locally on D11 and on D10.

No idea what the problem could be.

Any help will be greatly appreciated.

For now, I just disabled those 2 assertions when running on D10 in Gitlab CI.

Could you give it a try locally on D10 just to be sure we are not letting something go thru?

After generating some logs on Gitlab CI, I can see that the failing node doesn't seem to have the right domain source defined:

----------------------------------------
Path: /node/1
Active domain: example_com
Langcode: en
Url: route:entity.node.canonical;node=1
Entity id: 1
Source id: example_com

Found the problem after a few hours of debugging.

The test relies on the order of the results of call to loadMultiple().

For some reason, the order is different on Gitlab than locally and the test fails.

Don't know for what reason it has been done like this.

@idebr everything is green again.

Do you want to have a look before I merge?

Changes look good. However, there is still an open thread in the merge request that is possibly outdated?

It's merged. Thanks!