Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

Add Acquia-purge config for domain detection #487

Merged
merged 3 commits into from
Apr 11, 2018
Merged

Conversation

janette
Copy link
Member

@janette janette commented Mar 27, 2018

connects https://github.com/NuCivic/cadepttech/issues/107

https://docs.acquia.com/articles/acquia-purge-domain-detection

QA Tests

Change is applied to CDT Dev
run drush @cadepttech.dev ap-domains to verify only one domain is being purged.

switch ($_ENV['AH_SITE_ENVIRONMENT']) {
case 'prod':
$domain = $conf['acquia']['prod']['base_url'];
$domain = substr($domain, 7);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@janette It is possible that the base_urls will never be given in a different format, but I rather we do not make that assumption. I believe substr($domain, 7); is removing http:// from the base_url, but that feels very fragile. Maybe we should split the screen on "://" and take the second chunk, that will make no assumption about the protocol.

@fmizzell fmizzell merged commit e0ed310 into master Apr 11, 2018
@acouch acouch deleted the 107-acquia-purge branch May 4, 2018 13:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants