Uploaded image for project: 'WildFly'
  1. WildFly
  2. WFLY-9433

Apply InfiniteOrPositiveValidators to messaging attributes allow valid value -1

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Done
    • Icon: Major Major
    • 12.0.0.Beta1, 12.0.0.Final
    • 11.0.0.CR1
    • JMS
    • None

      There are several attributes defined from ActiveMQ Artemis configuration parameters in ActiveMQClient.java and ActiveMQDefaultConfiguration.java are allowed to take value -1.

      As they miss proper InfiniteOrPositiveValidators, they can also take any negative values which is incorrect.

      ActiveMQClient.java
      public static final long DEFAULT_CLIENT_FAILURE_CHECK_PERIOD_INVM = -1;
      public static final long DEFAULT_CONNECTION_TTL_INVM = -1;
      public static final int DEFAULT_CONSUMER_MAX_RATE = -1;
      public static final int DEFAULT_CONFIRMATION_WINDOW_SIZE = -1;
      public static final int DEFAULT_PRODUCER_MAX_RATE = -1;
      public static final int DEFAULT_THREAD_POOL_MAX_SIZE = -1;
      
      
      ActiveMQDefaultConfiguration.java
      private static long DEFAULT_CONNECTION_TTL_OVERRIDE = -1;
      private static int DEFAULT_JOURNAL_POOL_FILES = -1;
      private static long DEFAULT_CONNECTION_TTL_OVERRIDE = -1;
      private static int DEFAULT_JOURNAL_PERF_BLAST_PAGES = -1;
      private static long DEFAULT_SERVER_DUMP_INTERVAL = -1;
      private static long DEFAULT_MEMORY_MEASURE_INTERVAL = -1;
      private static int DEFAULT_BRIDGE_INITIAL_CONNECT_ATTEMPTS = -1;
      private static int DEFAULT_BRIDGE_RECONNECT_ATTEMPTS = -1;
      private static int DEFAULT_BRIDGE_PRODUCER_WINDOW_SIZE = -1;
      private static int DEFAULT_CLUSTER_INITIAL_CONNECT_ATTEMPTS = -1;
      private static int DEFAULT_CLUSTER_RECONNECT_ATTEMPTS = -1;
      private static int DEFAULT_HAPOLICY_BACKUP_REQUEST_RETRIES = -1;
      private static int DEFAULT_GROUPING_HANDLER_GROUP_TIMEOUT = -1;
      

              chaowan@redhat.com Chao Wang
              chaowan@redhat.com Chao Wang
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: