How does the output look like if there is no quota enabled/set?

There should be no line with [ aliases -> ... at all as far as I can see.

How does the output look like if there is no quota enabled/set?

Like in the provided example. ~ for no quota defined: * [email protected] ( 1,9M / ~ )

The aliases line is only shown when there are aliases configured.

@casperklein this PR introduced a regression as identified via setup email list output here:

* [email protected] ( 0 / ~ ) [0%]
    [ aliases -> [email protected], @localdomain2.com]

* [email protected] ( 0 / ~ ) [0%]

* [email protected] ( 0 / ~ ) [0%]

doveadm(# this is a test comment, please don't delete me :'(): Error: User doesn't exist
/usr/local/bin/listmailuser: line 12: 1024 *  : syntax error: operand expected (error token is "*  ")
/usr/local/bin/listmailuser: line 12: 1024 *  : syntax error: operand expected (error token is "*  ")
* # this is a test comment, please don't delete me :'( (  /  ) [%]
    [ aliases -> #]

doveadm(# this is also a test comment, :O): Error: User doesn't exist
/usr/local/bin/listmailuser: line 12: 1024 *  : syntax error: operand expected (error token is "*  ")
/usr/local/bin/listmailuser: line 12: 1024 *  : syntax error: operand expected (error token is "*  ")
* # this is also a test comment, :O (  /  ) [%]
    [ aliases -> #]

Comment support for this utility was previously implemented via this PR with the line:

(grep -v "^\s*$\|^\s*\#" "$DATABASE" || true) | awk -F '|' '{ print $1; }'

But your changes appear to have broken that? I think we could add a test (I don't mind writing the test), to check that the output doesn't contain the comments to avoid this in future?

This regression should be an easy fix though, right? Just re-adding the grep?

This regression should be an easy fix though, right? Just re-adding the grep?

I assume so?

I am working on a PR to merge for check-for-changes.sh after existing ones are merged to remove it's copies of other code that's fallen out of sync. One of the lines has similar grep to filter out comments, I extracted that out into a function instead into helper-functions.sh:

# Remove string input with only whitespace or `#` as the first non-whitespace character.
function _strip_comments
{
  grep -q -E "^\s*$|^\s*#" <<< "${1}"
}

This regression should be an easy fix though, right? Just re-adding the grep?

I assume so?

I am working on a PR to merge for check-for-changes.sh after existing ones are merged to remove it's copies of other code that's fallen out of sync. One of the lines has similar grep to filter out comments, I extracted that out into a function instead into helper-functions.sh:

# Remove string input with only whitespace or `#` as the first non-whitespace character.
function _strip_comments
{
  grep -q -E "^\s*$|^\s*#" <<< "${1}"
}

I like it 👍🏼