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

Comments

bdragon created an issue. See original summary.

bdragon’s picture

bdragon commented
Status: Active » Needs review
StatusFileSize
new 2958897-fix-hosting-import.patch570 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

helmo commented
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

bdragon commented

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.