evgeny/modoboa-installer
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed dist_info() case sensitivity for certbot installation

Browse Source 
On Debian 10, dist_info() returns a lower case "debian" and the installation of certbot fails.
I changed the installation method to check the distribution name in case insensitive way.
This commit is contained in:
Daniel Leicht
2020-10-28 15:30:09 +02:00
committed by GitHub
parent 619dc9f5e5
commit 85e76c1981
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
modoboa_installer/ssl.py
View File

@@ -75,7 +75,8 @@ class LetsEncryptCertificate(CertificateBackend):
def install_certbot(self): def install_certbot(self):
"""Install certbot script to generate cert.""" """Install certbot script to generate cert."""
name, version, _id = utils.dist_info() name, version, _id = utils.dist_info()
if name == "Ubuntu": name = name.lower()
if name == "ubuntu":
package.backend.update() package.backend.update()
package.backend.install("software-properties-common") package.backend.install("software-properties-common")
utils.exec_cmd("add-apt-repository -y universe") utils.exec_cmd("add-apt-repository -y universe")
@@ -83,10 +84,10 @@ class LetsEncryptCertificate(CertificateBackend):
utils.exec_cmd("add-apt-repository -y ppa:certbot/certbot") utils.exec_cmd("add-apt-repository -y ppa:certbot/certbot")
package.backend.update() package.backend.update()
package.backend.install("certbot") package.backend.install("certbot")
elif name == "Debian": elif name == "debian":
package.backend.update() package.backend.update()
package.backend.install("certbot") package.backend.install("certbot")
elif "CentOS" in name: elif "centos" in name:
package.backend.install("certbot") package.backend.install("certbot")
else: else:
utils.printcolor("Failed to install certbot, aborting.", utils.RED) utils.printcolor("Failed to install certbot, aborting.", utils.RED)