Due to missing https_drush_context_import, importing a site preconfigured to have https enabled does not update the site node correctly.
Rolling a patch.

CommentFileSizeAuthor
#2 2958897-fix-hosting-import.patch570 bytesbdragon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bdragon created an issue. See original summary.

bdragon’s picture

Status: Active » Needs review
FileSize
570 bytes

I am not certain whether it is more correct to have it here or in hosting_https.drush.inc (where its partner hosting_https_hosting_site_context_options() already lives)

Either place seems like it would be fine.

helmo’s picture

Status: Needs review » Fixed

Thanks, I've committed it to hosting_https.drush.inc as the other modules also implement that hook there.

  • helmo committed e4ec480 on 7.x-3.x authored by bdragon
    Issue #2958897 by bdragon: import of context to site node does not...
bdragon’s picture

Ah, I think in that case the function name should change to hosting_https_drush_context_import() instead of https_drush_context_import(), right?

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.