ICEfaces
  1. ICEfaces
  2. ICE-5006

Limit individual users to a maximum number of views

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.2
    • Fix Version/s: 1.8.2-EE-Beta, 1.8.2-EE-GA, 1.8.3
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEfaces
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

      Description

      Individual users may create very large numbers of views, resulting in memory exhaustion on the server.

      A new limit has been added, configurable through com.icesoft.faces.concurrentViewLimit.

        Activity

        Hide
        Ken Fyten added a comment -

        Increased default concurrentViewLimit to 50 views, up from 20. This is to avoid any likely impact on existing deployments with legitimate use-cases that require a large number of open views per session.

        Marking FIXED.

        Show
        Ken Fyten added a comment - Increased default concurrentViewLimit to 50 views, up from 20. This is to avoid any likely impact on existing deployments with legitimate use-cases that require a large number of open views per session. Marking FIXED.
        Hide
        Ted Goddard added a comment -

        The parameter has been renamed to "concurrentViewLimit". Additionally, a WARN log entry is now produced with the current view limit and the guilty sessionID.

        Show
        Ted Goddard added a comment - The parameter has been renamed to "concurrentViewLimit". Additionally, a WARN log entry is now produced with the current view limit and the guilty sessionID.
        Hide
        Ted Goddard added a comment -

        How about

        com.icesoft.faces.concurrentViewLimit

        Default Value 20

        Specifies to the ICEfaces framework how many concurrent views (typically one per browser window or tab) are allowed per user. When the limit is exceeded, a RuntimeException is thrown.

        Show
        Ted Goddard added a comment - How about com.icesoft.faces.concurrentViewLimit Default Value 20 Specifies to the ICEfaces framework how many concurrent views (typically one per browser window or tab) are allowed per user. When the limit is exceeded, a RuntimeException is thrown.
        Hide
        Ken Fyten added a comment -

        Can we change the config. param name to be more descriptive?

        Suggest: "com.icesoft.faces.activeViewLimit"

        Also, please add a description for the new config. param along the lines of the other params (http://www.icefaces.org/docs/v1_8_2/htmlguide/devguide/appendixB.html#1048527). Please explain or link to a reference around when views are created, destroyed so folks can understand the implications of this setting better.

        Show
        Ken Fyten added a comment - Can we change the config. param name to be more descriptive? Suggest: "com.icesoft.faces.activeViewLimit" Also, please add a description for the new config. param along the lines of the other params ( http://www.icefaces.org/docs/v1_8_2/htmlguide/devguide/appendixB.html#1048527 ). Please explain or link to a reference around when views are created, destroyed so folks can understand the implications of this setting better.

          People

          • Assignee:
            Unassigned
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: