Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
In this change: http://drupalcode.org/project/search_api_acquia.git/blobdiff/3ef80e5cb5f...
$search_host went from having $subscription['heartbeat_data']['search_service_colony'] take priority when set
$search_host = variable_get('acquia_search_host', 'search.acquia.com');
if (!empty($subscription['heartbeat_data']['search_service_colony'])) {
$search_host = $subscription['heartbeat_data']['search_service_colony'];
}
To having it only take priority if the variable acquia_search_host is empty.
$search_override_host = variable_get('acquia_search_host', '');
if (!empty($subscription['heartbeat_data']['search_service_colony']) && empty($search_override_host)) {
$search_host = $subscription['heartbeat_data']['search_service_colony'];
}
else {
// Get our override if we have one. Otherwise use the default.
$search_host = variable_get('acquia_search_host', 'search.acquia.com');
}
I believe this is incorrect, at least for our subscription it is always returning search.acquia.com which results in a 404 when indexing. When reverting to the beta3 version a different host is returned and indexing works correctly.
Comment | File | Size | Author |
---|---|---|---|
#1 | search_api_acquia-acquia_search_host_precedence-2087141-1.patch | 1.08 KB | tekante |
Comments
Comment #1
tekante CreditAttribution: tekante commentedPatch for restoring previous precedence order
Comment #2
kostajh CreditAttribution: kostajh commented@tekante thanks! This patch worked for me.
Comment #3
cpliakas CreditAttribution: cpliakas commentedAwesome. Will test this out right now.
Thanks for the contribution!
Comment #4
Nick_vhLogical and works as expected. Committing and giving you credit where credit is due :)