Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP]Add alias flag for alias-name support in wp #149

Closed
wants to merge 2 commits into from

Conversation

sagarnasit
Copy link
Contributor

@sagarnasit sagarnasit commented Jan 18, 2019

@sagarnasit sagarnasit changed the title Add alias flag for alias-name support in wp [WIP]Add alias flag for alias-name support in wp Jan 18, 2019
@@ -510,6 +517,16 @@ private function configure_site_files() {
$custom_conf_source = SITE_WP_TEMPLATE_ROOT . '/config/nginx/user.conf.mustache';
$process_user = posix_getpwuid( posix_geteuid() );

if ( ! empty( $this->site_data['site_aliases'] ) ) {

$domains = explode( ',', $this->site_data['site_aliases'] );
Copy link
Member

@mrrobot47 mrrobot47 Jan 18, 2019

Choose a reason for hiding this comment

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

explode only if ( 'subdom' === $this->site_data['app_sub_type'] ), otherwise directly use the input. No need for extra processing of exploding and then rejoining for no reason in other WordPress site types.

@dtbaker
Copy link

dtbaker commented Jan 28, 2019

Nice!!

Can we please ensure this works with multiple Letsencrypt wildcards, e.g.

sudo ee site create example.com --type=php --ssl=le --wildcard --with-db --alias=mysite.com,othersite.com

☝️ this should hopefully register a wildcard letsencrypt cert with all 3 on it: *.example.com,*.mysite.com,*.othersite.com

Cannot wait to try this out!

Thanks!!!

@kirtangajjar
Copy link
Contributor

Closing this PR as alias domain support has already been added in 4.1. However, it only worked on wpsubdom site. EasyEngine/site-command#341 fixes it and in next version, we'll be able to run alias domains in all the site types :)

Further updates will be posted on EasyEngine/feature-requests#23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Alias or Addon Domains for V4
4 participants