what's wrong with this one line filter

[Oink]

Originally posted by Bruce:
Replace the "exit 1" part with just "false", which is in the path.

condredirect aaa@aaa.aaa sh -c 'iftocc `tr "\n" " " < goodto.txt` || false'
and
condredirect aaa@aaa.aaa sh -c 'iftocc `tr "\n" " " < goodto.txt` | false'

both generate "command not found error" when trying to save filter. Pardon my not understanding the | symbol. It is a pipe, no?

condredirect aaa@aaa.aaa sh -c 'iftocc `tr "\n" " " < goodto.txt` false'

saves ok, but email failing the filter again bounces with

condredirect: fatal: permanent child error
vdeliver: Execution of vdeliver-predeliver failed

(Testing out of curiosity, I can definitely wait till the removal of mentioned patch is completed. I really appreciate your effort here, Bruce)

[sheila]

The double-pipe
||

is a symbol for "or". He's trying to make it so that if the iftocc fails, that the 'other' command that he is pasting onto the end is the value returned for the test, so that you can get around the bad value returned when iftocc fails. I'm not sure if shell scripts (about which I know very, very little) use short-circuit evaluation, but if they do, the second (or 'other' value) such as 'false' would only be returned in the case that the first value evaluates to failure.

By the way, I'm glad to see that someone presented this problem in a way that has caused it to be seriously looked at and re-evaluated. Somewhere else in the forums, I also publicly claimed that the condredirect was broken. (I tried to find it the other night by searching the forums, but had no success.) Anyhow, it didn't seem to draw the attention that this thread has drawn. I guess I didn't present the problem in a way that made it look like someone needed to 'fix' something?

Thanks to Bruce for checking it out and patching it (when he does get around to it ).

[kitchin]

Just to be clear, do you mean that "info condredirect" is now correct (except for the references to ".qmail"):

If program exits 0, condredirect
forwards the mail message to newaddress, and then exits
99, so further commands in .qmail are ignored.

If program exits 111, condredirect exits 111, so delivery
will be retried later.

If program exits anything else (or does not exist),
conÂ*dredirect exits 0, so the rest of .qmail will be processed
as usual.

But a paragraph should be added saying:

If program cannot be executed, condredirect exits N.

With the patch, N=100, but now N=0. Is that correct?

By the way, the info page for "iftocc" is very interesting. Looks like a great way to filter out BCC mail! Thanks...

[kitchin]

One more paragraph should be added, or rather inserted:

If program exits 100, condredirect exits 100, so the message
will bounce.

This is the undocumented behavior I have observed.
It's not really a quote, I am just using that tag.

[Oink]

Originally posted by Bruce:
The change is to remove this previous patch that is causing problems. [/i][/color]

Any progress?

[Bruce]

The problem with condredirect has been eliminated as a side effect the recent email system upgrades.