>>>>> On Tue, 05 Jan 2021, Michał Górny wrote:

> +     # check for the override
> +     local override_name=${ACCT_USER_NAME^^}
> +     local override_var=ACCT_USER_${override_name//-/_}_ID

I wonder about this line. Both hyphen and underscore are valid
characters in user names.

So, ACCT_USER_FOO_BAR_ID would override the id for both foo_bar and
foo-bar users.

Ulrich

Attachment: signature.asc
Description: PGP signature

Reply via email to