when run in a while loop, it appears to gobble up stdin

[dotysan]

And doesn't honor the args. Here's how to reproduce.

First it works on the cli.

 asn -c .XX

No countries matching '.XX'

But not in a while loop.

echo 'XX' |while read cc ; do echo "cc: $cc" ; asn -c ".$cc" ; done
cc: XX

<PrintUsage() output>

Error: no target specified

[dotysan]

The bug appears to be in this screwball logic.

asn/asn

Lines 4391 to 4405 in 2355d29

	if [ -t 0 ] || [ "$IS_HEADLESS" = true ] || [ -n "$PARALLEL_SEQ" ] || [ -n "$PARALLEL_PID" ]; then
	# input is from the terminal (or this is a headless instance)
	if [ "$OPTIONS_PRESENT" = false ]; then
	# shellcheck disable=SC2124
	userinput="$@"
	elif [ -z "$userinput" ] && [ "$SERVER_MODE" = false ]; then
	# an option was passed, but it was not one that requires a param (e.g. -m for monochrome mode)
	# fetch the actual target
	GetFullParamsFromCurrentPosition "$@"
	fi
	else
	# script was invoked with input from stdin (e.g. "asn -s < iplist")
	# read IP list from stdin and trim blanks and comments
	userinput=$(cat | grep -Ev '^$|^[ \t]*#')
	fi

[dotysan]

Should it check $OPTIONS_PRESENT before bailing on [ -t 0 ] and then gobbling up stdin?

[nitefood]

Hey @dotysan,

thanks for reporting. Option parsing logic needs some rework indeed, I haven't touched it in years.

In the meantime, as a workaround:

❯ echo 'XX' |while read cc ; do echo "cc: $cc" ; asn -c -- <<<".$cc" ; done
cc: XX

No countries matching '.XX'

[dotysan]

Nice trick! I too, was able to workaround with: for cc in XX ; do asn -c ".$cc" ; done since without the |while there's no subshell. I supposed process substitution would work too. while read -r cc ; do asn -c ".$cc" ; done < <(echo XX)

[dotysan]

And definitely related. #88