Those are genuine MoulinetteError because base on moulinette helpers

This commit is contained in:
Alexandre Aubin 2018-12-13 15:40:00 +00:00
parent f6b244f294
commit fb010765bd
2 changed files with 6 additions and 5 deletions

View file

@ -29,6 +29,7 @@ import json
import yaml import yaml
from moulinette import m18n, msettings from moulinette import m18n, msettings
from moulinette.core import MoulinetteError
from yunohost.utils.error import YunohostError from yunohost.utils.error import YunohostError
from moulinette.utils.log import getActionLogger from moulinette.utils.log import getActionLogger
@ -76,7 +77,7 @@ def domain_add(operation_logger, auth, domain, dyndns=False):
try: try:
auth.validate_uniqueness({'virtualdomain': domain}) auth.validate_uniqueness({'virtualdomain': domain})
except YunohostError: except MoulinetteError:
raise YunohostError('domain_exists') raise YunohostError('domain_exists')
operation_logger.start() operation_logger.start()

View file

@ -32,13 +32,13 @@ import base64
import subprocess import subprocess
from moulinette import m18n from moulinette import m18n
from yunohost.utils.error import YunohostError from moulinette.core import MoulinetteError
from moulinette.utils.log import getActionLogger from moulinette.utils.log import getActionLogger
from moulinette.utils.filesystem import read_file, write_to_file, rm from moulinette.utils.filesystem import read_file, write_to_file, rm
from moulinette.utils.network import download_json from moulinette.utils.network import download_json
from moulinette.utils.process import check_output from moulinette.utils.process import check_output
from yunohost.utils.error import YunohostError
from yunohost.domain import _get_maindomain, _build_dns_conf from yunohost.domain import _get_maindomain, _build_dns_conf
from yunohost.utils.network import get_public_ip from yunohost.utils.network import get_public_ip
from yunohost.log import is_unit_operation from yunohost.log import is_unit_operation
@ -74,7 +74,7 @@ def _dyndns_provides(provider, domain):
# Dyndomains will be a list of domains supported by the provider # Dyndomains will be a list of domains supported by the provider
# e.g. [ "nohost.me", "noho.st" ] # e.g. [ "nohost.me", "noho.st" ]
dyndomains = download_json('https://%s/domains' % provider, timeout=30) dyndomains = download_json('https://%s/domains' % provider, timeout=30)
except YunohostError as e: except MoulinetteError as e:
logger.error(str(e)) logger.error(str(e))
raise YunohostError('dyndns_could_not_check_provide', domain=domain, provider=provider) raise YunohostError('dyndns_could_not_check_provide', domain=domain, provider=provider)
@ -101,7 +101,7 @@ def _dyndns_available(provider, domain):
try: try:
r = download_json('https://%s/test/%s' % (provider, domain), r = download_json('https://%s/test/%s' % (provider, domain),
expected_status_code=None) expected_status_code=None)
except YunohostError as e: except MoulinetteError as e:
logger.error(str(e)) logger.error(str(e))
raise YunohostError('dyndns_could_not_check_available', raise YunohostError('dyndns_could_not_check_available',
domain=domain, provider=provider) domain=domain, provider=provider)