“Distribution for origin x not found”

Bug is here: https://github.com/W3EDGE/w3-total-cache/blob/c5aeabd74ff9899139bb16cbb314377b1a040229/CdnEngine_CloudFront.php

private function _get_distribution( $dists = null ) {
	[...]
	$dist = false;
	$origin = $this->_get_origin();
	$items = $dists['DistributionList']['Items'];
	foreach ( $items as $dist ) {
		if ( isset( $dist['Origins']['Items'][0]['DomainName'] ) &&
			 $dist['Origins']['Items'][0]['DomainName'] == $origin ) {
			return $dist;
		}
	}
	throw new \Exception( sprintf( 'Distribution for origin "%s" not found.', $origin ) );

$dist['Origins']['Items'][0]['DomainName'] should check for each Origin instead of only [0]. I changed it to [1] and “Test passed”. (I have two origins.)

I don’t think this does anything other than confirm that the distribution exists for said origin, though.

Ref: https://docs.aws.amazon.com/cloudfront/latest/APIReference/API_ListDistributions.html

Hello @jpb10,

I am sorry for the issue you are having with S3 Origins.

Just to confirm, when you changed $dist['Origins']['Items'][0]['DomainName'] to $dist['Origins']['Items'][1]['DomainName'] in CdnEngine_CloudFront.php, you are no longer experiencing the issue?

Correct.

The original code only looks at the first distribution.

Which is why the test passes when you first create the bucket and distribution thru W3TC, but fails after you add other distributions.

Example: https://forums.aws.amazon.com/thread.jspa?threadID=65546

Hello @jpb10,
Thank you for the information. We are going to check and test this and see if we can confirm this as a bug.
Thank you for your patience

Hello @jpb10,
This has issue has been resolved. Please check our public repo and the issue you opened there.

https://github.com/W3EDGE/w3-total-cache/issues/28