Fix some sed -r, and clean up some variable references ("$1" -> "$varname")

This commit is contained in:
stilez 2019-11-05 18:50:39 +00:00 committed by GitHub
parent 896778cead
commit cbacc779fc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -100,7 +100,7 @@ dns_pleskxml_add() {
return 1 return 1
fi fi
recid="$(_value "$results" | grep '<id>[0-9]\{1,\}</id>' | sed -r 's/^.*<id>([0-9]+)<\/id>.*$/\1/')" recid="$(_value "$results" | grep '<id>[0-9]\{1,\}</id>' | sed 's/^.*<id>\([0-9]\{1,\}\)<\/id>.*$/\1/')"
_info "Success. TXT record appears to be correctly added (Plesk record ID=$recid). Exiting dns_pleskxml_add()." _info "Success. TXT record appears to be correctly added (Plesk record ID=$recid). Exiting dns_pleskxml_add()."
@ -145,16 +145,16 @@ dns_pleskxml_rm() {
return 1 return 1
fi fi
_debug "Got list of DNS TXT records for root domain '$root_domain_name'"':\n'"$reclist" _debug "Got list of DNS TXT records for root domain '$root_domain_name'. Full list is:"'\n'"$reclist"
_debug "DNS TXT records for host '$fulldomain':"'\n'"$(_value "$reclist" | grep "<host>${fulldomain}.</host>")"
recid="$(_value "$reclist" \ recid="$(_value "$reclist" \
| grep "<host>$1.</host>" \ | grep "<host>${fulldomain}.</host>" \
| grep "<value>$txtvalue</value>" \ | grep "<value>${txtvalue}</value>" \
| sed -r 's/(^.*<id>|<\/id>.*$)//g' | sed 's/^.*<id>\([0-9]\{1,\}\)<\/id>.*$/\1/'
)" )"
_debug "List of DNS TXT records for host:"'\n'"$(_value "$reclist" | grep "<host>$1.</host>")"
if ! _value "$recid" | grep '^[0-9]\{1,\}$' >/dev/null; then if ! _value "$recid" | grep '^[0-9]\{1,\}$' >/dev/null; then
_err "DNS records for root domain '${root_domain_name}' (Plesk ID ${root_domain_id}) + host '${sub_domain_name}' do not contain the TXT record '${txtvalue}'" _err "DNS records for root domain '${root_domain_name}' (Plesk ID ${root_domain_id}) + host '${sub_domain_name}' do not contain the TXT record '${txtvalue}'"
_err "Cannot delete TXT record. Exiting." _err "Cannot delete TXT record. Exiting."
@ -341,13 +341,13 @@ _credential_check() {
# See notes at top of this file # See notes at top of this file
_pleskxml_get_root_domain() { _pleskxml_get_root_domain() {
_debug "Identifying DNS root domain for '$1' that is managed by the Plesk account."
original_full_domain_name="$1" original_full_domain_name="$1"
root_domain_name="$1"
_debug "Identifying DNS root domain for '$original_full_domain_name' that is managed by the Plesk account."
# test if the domain as provided is valid for splitting. # test if the domain as provided is valid for splitting.
if ! _countdots "$root_domain_name"; then if ! _countdots "$original_full_domain_name"; then
_err "Invalid domain. The ACME domain must contain at least two parts (aa.bb) to identify a domain and tld for the TXT record." _err "Invalid domain. The ACME domain must contain at least two parts (aa.bb) to identify a domain and tld for the TXT record."
return 1 return 1
fi fi
@ -372,6 +372,8 @@ _pleskxml_get_root_domain() {
# loop and test if domain, or any parent domain, is managed by Plesk # loop and test if domain, or any parent domain, is managed by Plesk
# Loop until we don't have any '.' in the string we're testing as a candidate Plesk-managed domain # Loop until we don't have any '.' in the string we're testing as a candidate Plesk-managed domain
root_domain_name="$original_full_domain_name"
while true; do while true; do
_debug "Checking if '$root_domain_name' is managed by the Plesk server..." _debug "Checking if '$root_domain_name' is managed by the Plesk server..."