James Server
  1. James Server
  2. JAMES-679

Make sure our container use an expiration for cached dns data

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.3.0
    • Fix Version/s: 2.3.1
    • Component/s: None
    • Labels:
      None

      Description

      Noel J. Bergman wrote:
      > Stefano Bagnara wrote:
      >> but it could happen (or maybe it already happen) that some third party
      >> library we use still make use of InetAddress "flawed" methods, and the
      >> above property would save us from the possible OutOfMemory because of
      >> the unbounded cache.
      >
      > Perhaps, but I would rather vet the code, and be more careful about what we use. Now that we know one of the things to look for carefully, we can avoid it. The better solution is to avoid the problem, not mask it.
      >
      > — Noel

      I agree: we avoid it in trunk because of this.

      About the "not masking" I think that even if we solved it we should better add the property because our users could easily make our mistakes and use InetAddress in their mailets. The fact that DNSServer is not part of the mailet apis incentivate them to use InetAdress and not the James Server specific service.

      So I think we should really add the ttl property.

      Either "-Dsun.net.inetaddr.ttl=10" in the command line or
      java.security.Security.setProperty("networkaddress.cache.ttl" , "10");
      Should work.

        Issue Links

          Activity

          Stefano Bagnara created issue -
          Stefano Bagnara made changes -
          Field Original Value New Value
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 2.3.1-dev [ 12312150 ]
          Resolution Fixed [ 1 ]
          Fix Version/s Next Minor [ 12312136 ]
          Fix Version/s Trunk [ 12312135 ]
          Noel J. Bergman made changes -
          Status Resolved [ 5 ] Reopened [ 4 ]
          Resolution Fixed [ 1 ]
          Stefano Bagnara made changes -
          Link This issue relates to JAMES-774 [ JAMES-774 ]
          Stefano Bagnara made changes -
          Fix Version/s 2.3.1-dev [ 12312150 ]
          Status Reopened [ 4 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Fix Version/s Next Major [ 10427 ]
          Danny Angus made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Mark Thomas made changes -
          Workflow jira [ 12388797 ] Default workflow, editable Closed status [ 12566886 ]
          Mark Thomas made changes -
          Workflow Default workflow, editable Closed status [ 12566886 ] jira [ 12581931 ]

            People

            • Assignee:
              Stefano Bagnara
              Reporter:
              Stefano Bagnara
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development