benningm / mtpolicyd

a modular policy daemon for postfix
21 stars 3 forks source link

Problems with = sign in output #1

Closed jahlives closed 10 years ago

jahlives commented 10 years ago

In debian package there seems to b e a problem with the output of =-signs in the answers from mtpolicyd on the CLI. For example with SPF plugin which has a default_authority_explanation like

reject_message = "Mail rejected by SPF. See %AUTH_EXPL%" default_authority_explanation = "http:\/\/www.openspf.net/Why?mfrom=s;id=%{S};ip=%{I};r=%{R}"

which lead to the following output on CLI when testing with policyd-client

reject Mail rejected by SPF. See http:\/\/www.openspf.net/Why?mfrom

seems it breaks with the first equal sign. In the logfile the output is correct

result=reject Mail rejected by SPF. See http:\/\/www.openspf.net/Why?mfrom=s;id=bob%40brain-force.ch;ip=213.196.149.40;r=mail2.brain-force.ch

backslashes added in order that no clickable URL is generated. Backslashes do not occur in my config or output!

benningm commented 10 years ago

Seems like a parsing bug in the client. Fixed it in 01a20c8674281a3ed22ec7c243e0fcb181a10ad5. Fixed it also in 2 other places.

benningm commented 10 years ago

The bug is fixed in 1.13. It may take a few hours till the new version reached CPAN and the yum/apt repositories.