[tomcat] 01/03: Fix IDE warnings

[markt]

This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 8.5.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git

commit bb0df62ab311d8f264d88aa7c2ec91285a68d7ac
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue May 9 20:32:29 2023 +0100

    Fix IDE warnings
---
 java/org/apache/catalina/filters/RateLimitFilter.java     | 10 ++++++----
 java/org/apache/catalina/util/TimeBucketCounter.java      |  1 +
 test/org/apache/catalina/filters/TestRateLimitFilter.java |  9 ++++-----
 3 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/java/org/apache/catalina/filters/RateLimitFilter.java 
b/java/org/apache/catalina/filters/RateLimitFilter.java
index 32262b8b2a..f3ded50cf0 100644
--- a/java/org/apache/catalina/filters/RateLimitFilter.java
+++ b/java/org/apache/catalina/filters/RateLimitFilter.java
@@ -200,8 +200,9 @@ public class RateLimitFilter implements Filter {
         actualRequests = (int) Math.round(bucketCounter.getRatio() * 
bucketRequests);
 
         log.info(sm.getString("rateLimitFilter.initialized",
-            filterName, bucketRequests, bucketDuration, getActualRequests(),
-            getActualDurationInSeconds(), (!enforce ? "Not " : "") + 
"enforcing")
+            filterName, Integer.valueOf(bucketRequests), 
Integer.valueOf(bucketDuration),
+            Integer.valueOf(getActualRequests()), 
Integer.valueOf(getActualDurationInSeconds()),
+            (!enforce ? "Not " : "") + "enforcing")
         );
     }
 
@@ -212,13 +213,14 @@ public class RateLimitFilter implements Filter {
         String ipAddr = request.getRemoteAddr();
         int reqCount = bucketCounter.increment(ipAddr);
 
-        request.setAttribute(RATE_LIMIT_ATTRIBUTE_COUNT, reqCount);
+        request.setAttribute(RATE_LIMIT_ATTRIBUTE_COUNT, 
Integer.valueOf(reqCount));
 
         if (enforce && (reqCount > actualRequests)) {
 
             ((HttpServletResponse) response).sendError(statusCode, 
statusMessage);
             log.warn(sm.getString("rateLimitFilter.maxRequestsExceeded",
-                filterName, reqCount, ipAddr, getActualRequests(), 
getActualDurationInSeconds())
+                filterName, Integer.valueOf(reqCount), ipAddr, 
Integer.valueOf(getActualRequests()),
+                Integer.valueOf(getActualDurationInSeconds()))
             );
 
             return;
diff --git a/java/org/apache/catalina/util/TimeBucketCounter.java 
b/java/org/apache/catalina/util/TimeBucketCounter.java
index 5c602c96af..d104eebf4e 100644
--- a/java/org/apache/catalina/util/TimeBucketCounter.java
+++ b/java/org/apache/catalina/util/TimeBucketCounter.java
@@ -191,6 +191,7 @@ public class TimeBucketCounter {
             this.sleeptime = sleeptime;
         }
 
+        @SuppressWarnings("sync-override")
         @Override
         public void start() {
             isRunning = true;
diff --git a/test/org/apache/catalina/filters/TestRateLimitFilter.java 
b/test/org/apache/catalina/filters/TestRateLimitFilter.java
index 9f82306367..684bd3b258 100644
--- a/test/org/apache/catalina/filters/TestRateLimitFilter.java
+++ b/test/org/apache/catalina/filters/TestRateLimitFilter.java
@@ -31,7 +31,6 @@ import org.junit.Assert;
 import org.junit.Test;
 
 import org.apache.catalina.Context;
-import org.apache.catalina.LifecycleException;
 import org.apache.catalina.filters.TestRemoteIpFilter.MockFilterChain;
 import org.apache.catalina.filters.TestRemoteIpFilter.MockHttpServletRequest;
 import org.apache.catalina.startup.Tomcat;
@@ -63,7 +62,7 @@ public class TestRateLimitFilter extends TomcatBaseTest {
         int allowedRequests = (int) 
Math.round(rateLimitFilter.bucketCounter.getRatio() * bucketRequests);
 
         long sleepTime = 
rateLimitFilter.bucketCounter.getMillisUntilNextBucket();
-        System.out.printf("Sleeping %d millis for the next time bucket to 
start\n", sleepTime);
+        System.out.printf("Sleeping %d millis for the next time bucket to 
start\n", Long.valueOf(sleepTime));
         Thread.sleep(sleepTime);
 
         TestClient tc1 = new TestClient(rateLimitFilter, filterChain, 
"10.20.20.5", 200, 5);
@@ -85,8 +84,7 @@ public class TestRateLimitFilter extends TomcatBaseTest {
         Assert.assertEquals(429, tc4.results[allowedRequests]);     // 
subsequent requests dropped
     }
 
-    private RateLimitFilter testRateLimitFilter(FilterDef filterDef, Context 
root)
-            throws LifecycleException, IOException, ServletException {
+    private RateLimitFilter testRateLimitFilter(FilterDef filterDef, Context 
root) throws ServletException {
 
         RateLimitFilter rateLimitFilter = new RateLimitFilter();
         filterDef.setFilterClass(RateLimitFilter.class.getName());
@@ -138,7 +136,8 @@ public class TestRateLimitFilter extends TomcatBaseTest {
                     response.setRequest(request);
                     filter.doFilter(request, response, filterChain);
                     results[i] = response.getStatus();
-                    System.out.printf("%s %s: %s %d\n", ip, new Date(), i + 1, 
response.getStatus());
+                    System.out.printf("%s %s: %s %d\n", ip, new Date(), 
Integer.valueOf(i + 1),
+                            Integer.valueOf(response.getStatus()));
                     Thread.sleep(sleep);
                 }
             }


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org