Skip to content

Commit 28f438a

Browse files
authored
Merge pull request #5111 from scruel/scruel-patch-1
2 parents 43b5ea8 + 9ff89b5 commit 28f438a

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

deploy/synology_dsm.sh

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,8 @@ synology_dsm_deploy() {
227227
fi
228228
fi
229229

230-
error_code=$(echo "$response" | grep '"error"' | grep -oP '(?<="code":)\d+')
230+
error_code=$(echo "$response" | grep '"error":' | grep '"code":[0-9]*' | grep -o '[0-9]*')
231+
_debug2 error_code "$error_code"
231232
# Account has 2FA-OTP enabled, since error 403 reported.
232233
# https://global.download.synology.com/download/Document/Software/DeveloperGuide/Firmware/DSM/All/enu/Synology_DiskStation_Administration_CLI_Guide.pdf
233234
if [ "$error_code" == "403" ]; then
@@ -261,7 +262,8 @@ synology_dsm_deploy() {
261262
_secure_debug2 SYNO_DEVICE_ID "$SYNO_DEVICE_ID"
262263
fi
263264
fi
264-
error_code=$(echo "$response" | grep '"error"' | grep -oP '(?<="code":)\d+')
265+
error_code=$(echo "$response" | grep '"error":' | grep '"code":[0-9]*' | grep -o '[0-9]*')
266+
_debug2 error_code "$error_code"
265267
fi
266268

267269
if [ -n "$error_code" ]; then
@@ -323,7 +325,8 @@ synology_dsm_deploy() {
323325
id=$(echo "$response" | sed -n "s/.*\"desc\":\"$escaped_certificate\",\"id\":\"\([^\"]*\).*/\1/p")
324326
_debug2 id "$id"
325327

326-
error_code=$(echo "$response" | grep '"error"' | grep -oP '(?<="code":)\d+')
328+
error_code=$(echo "$response" | grep '"error":' | grep '"code":[0-9]*' | grep -o '[0-9]*')
329+
_debug2 error_code "$error_code"
327330
if [ -n "$error_code" ]; then
328331
if [ "$error_code" -eq 105 ]; then
329332
_err "Current user is not administrator and does not have sufficient permission for deploying."

0 commit comments

Comments
 (0)