mirror of
https://github.com/Security-Onion-Solutions/securityonion.git
synced 2025-12-22 08:53:06 +01:00
Try retrying in curl instead of shell function
This commit is contained in:
@@ -328,10 +328,16 @@ retry() {
|
|||||||
run_check_net_err() {
|
run_check_net_err() {
|
||||||
local cmd=$1
|
local cmd=$1
|
||||||
local err_msg=${2:-"Unknown error occured, please check /root/$WHATWOULDYOUSAYYAHDOHERE.log for details."} # Really need to rename that variable
|
local err_msg=${2:-"Unknown error occured, please check /root/$WHATWOULDYOUSAYYAHDOHERE.log for details."} # Really need to rename that variable
|
||||||
|
local no_retry=$3
|
||||||
|
|
||||||
local exit_code
|
local exit_code
|
||||||
|
if [[ -z $no_retry ]]; then
|
||||||
retry 50 10 "$cmd"
|
retry 50 10 "$cmd"
|
||||||
exit_code=$?
|
exit_code=$?
|
||||||
|
else
|
||||||
|
eval "$cmd"
|
||||||
|
exit_code=$?
|
||||||
|
fi
|
||||||
|
|
||||||
if [[ $exit_code -ne 0 ]]; then
|
if [[ $exit_code -ne 0 ]]; then
|
||||||
echo "Command failed with error $exit_code"
|
echo "Command failed with error $exit_code"
|
||||||
|
|||||||
@@ -130,8 +130,9 @@ update_docker_containers() {
|
|||||||
|
|
||||||
# Let's make sure we have the public key
|
# Let's make sure we have the public key
|
||||||
run_check_net_err \
|
run_check_net_err \
|
||||||
"curl -sSL https://raw.githubusercontent.com/Security-Onion-Solutions/securityonion/master/KEYS -o $SIGNPATH/KEYS" \
|
"curl --retry 20 --retry-delay 30 -sSL https://raw.githubusercontent.com/Security-Onion-Solutions/securityonion/master/KEYS -o $SIGNPATH/KEYS" \
|
||||||
"Could not pull signature key file, please ensure connectivity to https://raw.gihubusercontent.com" >> "$LOG_FILE" 2>&1
|
"Could not pull signature key file, please ensure connectivity to https://raw.gihubusercontent.com" \
|
||||||
|
noretry >> "$LOG_FILE" 2>&1
|
||||||
result=$?
|
result=$?
|
||||||
if [[ $result -eq 0 ]]; then
|
if [[ $result -eq 0 ]]; then
|
||||||
cat $SIGNPATH/KEYS | gpg --import - >> "$LOG_FILE" 2>&1
|
cat $SIGNPATH/KEYS | gpg --import - >> "$LOG_FILE" 2>&1
|
||||||
@@ -157,8 +158,9 @@ update_docker_containers() {
|
|||||||
|
|
||||||
# Get signature
|
# Get signature
|
||||||
run_check_net_err \
|
run_check_net_err \
|
||||||
"curl -A '$CURLTYPE/$CURRENTVERSION/$OS/$(uname -r)' https://sigs.securityonion.net/$VERSION/$i:$VERSION$IMAGE_TAG_SUFFIX.sig --output $SIGNPATH/$image.sig" \
|
"curl --retry 20 --retry-delay 30 -A '$CURLTYPE/$CURRENTVERSION/$OS/$(uname -r)' https://sigs.securityonion.net/$VERSION/$i:$VERSION$IMAGE_TAG_SUFFIX.sig --output $SIGNPATH/$image.sig" \
|
||||||
"Could not pull signature file, please ensure connectivity to https://sigs.securityonion.net " >> "$LOG_FILE" 2>&1
|
"Could not pull signature file, please ensure connectivity to https://sigs.securityonion.net " \
|
||||||
|
noretry >> "$LOG_FILE" 2>&1
|
||||||
if [[ $? -ne 0 ]]; then
|
if [[ $? -ne 0 ]]; then
|
||||||
echo "Unable to pull signature file for $image" >> "$LOG_FILE" 2>&1
|
echo "Unable to pull signature file for $image" >> "$LOG_FILE" 2>&1
|
||||||
exit 1
|
exit 1
|
||||||
|
|||||||
Reference in New Issue
Block a user