James Server
  1. James Server
  2. JAMES-670

Per IP connection limiting is not configurable per service, nor is the configuration logged during initialization.

    Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0, Trunk
    • Fix Version/s: 3.0-M1
    • Component/s: None
    • Labels:
      None

      Description

      Per IP connection limiting is not configurable per service, nor is the configuration logged during initialization.

        Issue Links

          Activity

          Hide
          Norman Maurer added a comment -

          We now use ConnectionFilter todo this stuff in MINA. So I will close this..

          Show
          Norman Maurer added a comment - We now use ConnectionFilter todo this stuff in MINA. So I will close this..
          Hide
          Norman Maurer added a comment -

          Sure its possible.. I also thought about this before i did the changes.. But remember that whould also need changes to assembly.xml again etc... But if all agree i can refactor it..

          Show
          Norman Maurer added a comment - Sure its possible.. I also thought about this before i did the changes.. But remember that whould also need changes to assembly.xml again etc... But if all agree i can refactor it..
          Hide
          Stefano Bagnara added a comment -

          I see this feature changed more code than I thought.

          I would prefer it to be contained in a different class, is it possible?
          I also noticed that there is a lot of int<>Integer<>String conversions in ConnectionPerIpMap and this could be made much more performant by using only ints.

          Show
          Stefano Bagnara added a comment - I see this feature changed more code than I thought. I would prefer it to be contained in a different class, is it possible? I also noticed that there is a lot of int< >Integer< >String conversions in ConnectionPerIpMap and this could be made much more performant by using only ints.
          Hide
          Norman Maurer added a comment -

          I leave this open cause maybe it whould be a better solution to create a new connectionmanager and specify this connectionmanager in assembly.xml to add this feature..

          Show
          Norman Maurer added a comment - I leave this open cause maybe it whould be a better solution to create a new connectionmanager and specify this connectionmanager in assembly.xml to add this feature..
          Hide
          Stefano Bagnara added a comment -

          Marked as a new feature: it is different to limit connections per service or per application and the proposed change does not include the current behaviour.

          So, if anyone want to implement this improvement, please leave us also the old behaviour

          Show
          Stefano Bagnara added a comment - Marked as a new feature: it is different to limit connections per service or per application and the proposed change does not include the current behaviour. So, if anyone want to implement this improvement, please leave us also the old behaviour

            People

            • Assignee:
              Norman Maurer
              Reporter:
              Noel J. Bergman
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development