Custom exit message according to flag


 
Thread Tools Search this Thread
Top Forums Shell Programming and Scripting Custom exit message according to flag
# 1  
Old 03-12-2020
Custom exit message according to flag

Hello everyone,

I'm currently working on a script that uses curl's to check internet connectivity with and without a proxy being specified.
Basically what I'd like to achieve is having the script output fail or success according to the "method" used (direct or via proxy).

A very ugly way of doing it:
Code:
 # "curl_args" contains "-x $2" and gets set if $2 exists
curl ${curl_args} "$1" &> /dev/null

if [ $? == 0 ]; then
   if [ "$#" == 1 ]; then
      printf "OK - able to connect to $1\n"
      exit 0
   elif [ "$#" == 2 ]; then
      printf "OK - able to connect to $1 from $2\n"
      exit 0
   fi
else
   if [ "$#" == 1 ]; then
      printf "CRITICAL - unable to connect to $1\n"
      exit 1
   elif [ "$#" == 2 ]; then
      printf "CRITICAL - unable to connect to $1 from $2\n"
      exit 1
   fi

The script expects 2x arguments. The first one is the URL and the second one is the proxy address.

Whats a more dynamic way of writing that ugly if statement?
# 2  
Old 03-12-2020
Id like to see the code for the commented line, because the code would be shorter.

Other than that sounds more lile homework to me.
Is it?
# 3  
Old 03-12-2020
no it's not homework.

I've made some changes to it (whole code as requested):

Code:
# if we specify a proxy on $2, add the -x flag to curl
if [ "$proxy" ]; then
  # the whitespace is important
  curl_args+=" -x $2"
  exit_msg+=" via $proxy"
fi

usage() {
  printf "$(basename $0) URL [proxy]\n"
  exit 0
}

exit_ok() {
  printf "OK - $1"
  exit 0
}

exit_fail() {
  printf "FAILED - $1"
  exit 1
}

curl_cmd="curl ${curl_args} $url"
case $# in
  1) $curl_cmd &>/dev/null && exit_ok "$exit_msg" || exit_fail "$exit_msg"
     ;;
  2) $curl_cmd &>/dev/null && exit_ok "$exit_msg" || exit_fail "$exit_msg"
     ;;
  *) usage
     ;;
esac

I'm currently trying to get rid of exit_ok() and exit_fail()

Last edited by da1; 03-12-2020 at 04:15 PM..
# 4  
Old 03-12-2020
I didnt request the whole code, just the first commented line.
Anyway, thanks.

First and foremost, both "${2:-}" are useless, as that would fill an empty variable with... emptyness.
Second, you dont want something in a variable that defines an extra option - if it hasnt been set by the user.

I'll keep the short approach you've had in the first post:
Code:
curl_args="--fail --max-time 3"


case ${#@} in
1)	url="$1"
	exit_msg="connecting to $url"
	;;
2)	url="$1"
	curl_args="$curl_args -x \"$2\""	# The 'inner' quotes needs escape so they are there when the variable is reused
	exit_msg="connecting to $url via $2"
	;;
esac

curl $curl_args "$url"
ret=$?

[[ 0 = $ret ]] && \
	exit_msg="SUCCESS - $exit_msg" || \
	exit_msg="FAILURE - $exit_msg"

echo "$exit_msg"
exit $ret

hth & hf

Last edited by sea; 03-12-2020 at 03:10 PM.. Reason: typo
# 5  
Old 03-12-2020
hmm, very interesting construct. thank you for that.

One question though regarding the following piece of code:
Code:
[[ 0 = $ret ]] && \
	exit_msg="SUCCESS - $exit_msg" || \
	exit_msg="FAILURE - $exit_msg"

Why not do a proper if/else statement here?

Quote:
First and foremost, both "${2:-}" are useless, as that would fill an empty variable with... emptyness.
I did that because of the "set -u". What would be the right approach?

Quote:
Second, you dont want something in a variable that defines an extra option - if it hasnt been set by the user.
Ok, can you elabortate on that? Why is it wrong (?) to append a string to a variable?
# 6  
Old 03-12-2020
You're welcome.

Quote:
Originally Posted by da1
One question though regarding the following piece of code:
Code:
[[ 0 = $ret ]] && \
	exit_msg="SUCCESS - $exit_msg" || \
	exit_msg="FAILURE - $exit_msg"

Why not do a proper if/else statement here?
It's shorter and i'm lazy.
Also it only 'contains' 1 task per situation, so this is enough.
However, if there are more lines to be made for each situation, i'd go with a proper if/else statement as well.

Quote:
Originally Posted by da1
I did that because of the "set -u". What would be the right approach?
Ok, can you elabortate on that? Why is it wrong (?) to append a string to a variable?
If you want to check wether a variable exists (or has content) or not (empty) and DECIDE what to execute according to that, it's a bad choice to fill the variables anway.

One usualy uses this to make sure a REQUIRED variable gets it's DEFAULT or FALLBACK values, as in:
Code:
var=${1:-default-value}

I have.. uhm.. misspelled.. my... uhm.. concern... in the previous post, because:
${1:-} doesnt actualy do something, and is basicly the same as ${1}, but it (the first one) indicates that you wanted to put some variable there, which then might interfere.
Also, by setting a string to an empty variable, you'd say the script to ALWAYS use '$proxy', because '$proxy' would contain something. (well, not in your actual code, that's just bad syntax)

The set -u only requires you to declare the variable before invoking it.
So any of these are valid options for that case/command:
Quote:
declare var
local var
var=""
hth

--- Post updated at 21:54 ---

On a further note, your exit codes in those functions just exit the fuction, therefor return could be used there.

For example, try this:
Code:
my_msg(){ echo "${1:-some string i dont care} ; return 1 ; }
my_msg
ret=$?
echo $ret
exit $ret

And then change the return number.

Hth
This User Gave Thanks to sea For This Post:
Login or Register to Ask a Question

Previous Thread | Next Thread

10 More Discussions You Might Find Interesting

1. IP Networking

Insmod custom module fails with message : disagrees about version of symbol ...

Hello : I want to make a netfilter conntrack module for myself. So I copy all the source code about netfilter conntrack from kernel source tree to my external directory. It can be insmod after compiled. Then I add some members to the struct nf_conn, and it 's compiled successfully. However, it... (1 Reply)
Discussion started by: 915086731
1 Replies

2. Shell Programming and Scripting

awk to supress error message with custom text

After a bash function is run the below file is produced: out_name.txt tab-delimeted Input Errors and warnings AccNo Genesymbol Variant Reference Sequence Start Descr. Coding DNA Descr. Protein Descr. GeneSymbol Coding DNA Descr. GeneSymbol Protein Descr. Genomic... (3 Replies)
Discussion started by: cmccabe
3 Replies

3. UNIX for Dummies Questions & Answers

Get the id with specify flag

.... means multi line ddd,bug fgdrg dfdfsdfdfsd fsdfdfdfd fdfdsfdsfsd ....... flag2 ...... aaa,bug sfsfsfsfs dfdfsdfdfsd fsdfdfdfd fdfdsfdsfsd ...... flag1 ...... ddd,bug fgdrg dfdfsdfdfsd (9 Replies)
Discussion started by: yanglei_fage
9 Replies

4. Shell Programming and Scripting

Need the difference between exit 1 & exit 7

Hi In one of the script I am seeing some thing like exit 7,exit 1,exit 2,exit 3,exit 9,exit6.What is the difference between all of this exit.Can anyone help here please (3 Replies)
Discussion started by: ginrkf
3 Replies

5. UNIX for Dummies Questions & Answers

'h' flag in du

Hey, all! Why is the "human readable" flag changing the behavior of du? And while I'm at it, can you make du only look at files, not directories. I often find myself wanting to find the largest file(s) in a dir or vol. Using 'find' itself, it seems you have to at least be able to guess the size of... (2 Replies)
Discussion started by: sudon't
2 Replies

6. Shell Programming and Scripting

Printing all lines before a specific string and a custom message 2 lines after

Hello all, I need to print all the lines before a specific string and print a custom message 2 lines after that. So far I have managed to print everything up the string, inclusively, but I can't figure out how to print the 2 lines after that and the custom message. My code thus far is:... (4 Replies)
Discussion started by: SEinT
4 Replies

7. AIX

du flag -x

Hi, I would like to know if there's any option to use with the du command so that I can list only the files/directories on the current filesystem... I usually use du -gs *But I'd like to see only the directories in the filesystem I am on, and not the mount point directory of other fss... ... (6 Replies)
Discussion started by: Casey
6 Replies

8. Programming

How to limit max no of message in a posix message queue

Hii can anyone pls tell how to limit the max no of message in a posix message queue. I have made changes in proc/sys/fs/mqueue/msg_max But still whenever i try to read the value of max. message in the queue using attr.mq_curmsgs (where struct mq_attr attr) its giving the default value as 10.... (0 Replies)
Discussion started by: mohit3884
0 Replies

9. Shell Programming and Scripting

cant get the right exit message

#!/bin/ksh application task run command //returns 0 if successful if ; then echo "Ran Fine" else echo "Didnt run" fi When I run the script, here is the output Status code = 0 ksh: [0: not found. Job didnt run any suggestions? (4 Replies)
Discussion started by: bryan
4 Replies

10. UNIX for Dummies Questions & Answers

Where can I find a list of exit codes? (Exit code 64)

I'm receiving an exit code 64 in our batch scheduler (BMC product control-m) executing a PERL script on UX-HP. Can you tell me where I can find a list of exit codes and their meaning. I'm assuming the exit code is from the Unix operating system not PERL. (3 Replies)
Discussion started by: jkuchar747
3 Replies
Login or Register to Ask a Question