You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2005/09/30 21:17:39 UTC

[Bug 4611] New: Reverse DNS patch for 3.1.0

http://bugzilla.spamassassin.org/show_bug.cgi?id=4611

           Summary: Reverse DNS patch for 3.1.0
           Product: Spamassassin
           Version: 3.1.0
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: minor
          Priority: P5
         Component: Libraries
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: gurnec@gis.net


It looks like the Mail::SpamAssassin::DnsResolver module doesn't transform IP 
address lookups into in-addr.arpa PTR lookups as Net::DNS::Resolver does. 
lookup_ptr in Dns.pm expects it to...

A patch against 3.1.0 follows.

Regards,
Chris



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 4611] [review] Reverse DNS patch for 3.1.0

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4611





------- Additional Comments From jm@jmason.org  2005-12-19 08:24 -------
Created an attachment (id=3311)
 --> (http://issues.apache.org/SpamAssassin/attachment.cgi?id=3311&action=view)
patch as applied to trunk

bug 4735 changed the code a little bit, so I had to redo some of the patch.
depending on if this ever gets another vote (hint hint!), this alternative
version of the patch may be necessary in 3.1.0.

anyway, this is in trunk as r357659.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 4611] [review] Reverse DNS patch for 3.1.0

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4611


spamassassin@dostech.ca changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Status Whiteboard|                            |needs 1 vote






------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 4611] [review] Reverse DNS patch for 3.1.0

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4611


spamassassin@dostech.ca changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Reverse DNS patch for 3.1.0 |[review] Reverse DNS patch
                   |                            |for 3.1.0






------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 4611] Reverse DNS patch for 3.1.0

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4611





------- Additional Comments From spamassassin@dostech.ca  2005-10-18 00:27 -------
+1



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 4611] [review] Reverse DNS patch for 3.1.0

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4611


sidney@sidney.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Status Whiteboard|needs 1 vote                |






------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 4611] Reverse DNS patch for 3.1.0

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4611


jm@jmason.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|Undefined                   |3.1.1




------- Additional Comments From jm@jmason.org  2005-09-30 13:05 -------
+1.  oops



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 4611] Reverse DNS patch for 3.1.0

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4611





------- Additional Comments From gurnec@gis.net  2005-09-30 12:23 -------
Created an attachment (id=3155)
 --> (http://bugzilla.spamassassin.org/attachment.cgi?id=3155&action=view)
Patches DnsResolver for rdns


Patches Mail::SpamAssassin::DnsResolver (3.1.0) to do reverse lookups as
Net::DNS::Resolver does which fixes lookup_ptr.




------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 4611] [review] Reverse DNS patch for 3.1.0

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4611


jm@jmason.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |amal@krasn.ru




------- Additional Comments From jm@jmason.org  2005-12-19 18:49 -------
*** Bug 4743 has been marked as a duplicate of this bug. ***



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

[Bug 4611] [review] Reverse DNS patch for 3.1.0

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4611


sidney@sidney.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




------- Additional Comments From sidney@sidney.com  2005-12-19 09:49 -------
+1

It looks like Justin accidentally checked this patch in to the 3.1 branch along
with the patch for bug 4719 in commit revision 357661

Since my vote makes this ready to commit anyway, there's nothing to revert and
I'm closing this as fixed.




------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.