search

interbrite / letsencrypt-vesta

Automate Let's Encrypt Certificate Installation for VestaCP
Other
217 stars 64 forks source link

v-list-web-domains-alias removal in R16 #28

Closed thinkingcap closed 8 years ago

thinkingcap commented 8 years ago

Closes #27 for R16 This would break earlier releases though.

quacos commented 8 years ago

This works only when domain has just one alias. Aliases are space-separated, but certbot need them comma separated.

jpitoniak commented 8 years ago

Thanks for the suggestion. As @quacos noted, however, this didn't quite work because the aliases are listed space separated instead of comma separated. Instead, I found that v-get-web-domain-value "$1" "$2" ALIAS returns the list in a more usable format (comma separated and wit no additional grepping or cutting required), so I've updated the script to use that.