https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7649
Evan Booyens <evan.booy...@hetzner.co.za> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #9 from Evan Booyens <evan.booy...@hetzner.co.za> ---
(In reply to Henrik Krohns from comment #2)
> Doesn't seem like a feature that general population needs.
>
> Atleast there should be config options for it.
>
> Also it doesn't make any sense to parse Received headers again, as SA has
> already did that and the info is available in internals. Your code parses
> "anything that looks like IP" from any Received header, which doesn't make
> much sense. Do you actually want to see everything, or just perhaps the
> first external relay or all trusted relays etc, otherwise it's just full of
> faked IPs..
Hi - thanks for the feedback
I would prefer to extract this info from the internals as it saves on
re-parsing the email. Some pointers would be helpful, else I can just troll
through the code.
(In reply to Henrik Krohns from comment #8)
> Right thanks Steve, I don't much use spamd so it's a bit unfamiliar yet,
> set_spamd_result_item looks just like what is wanted here.
>
> Simply use these as required to get wanted IPs
>
> $pms->{relays_internal}
> $pms->{relays_external}
> $pms->{relays_trusted}
> $pms->{relays_untrusted}
Thanks for the fast response guys - I have something I can use now without
having to patch spamd, so we can close this as a patch request.
--
You are receiving this mail because:
You are the assignee for the bug.
