isapir commented on code in PR #767: URL: https://github.com/apache/tomcat/pull/767#discussion_r1828114910
########## java/org/apache/catalina/util/RateLimiter.java: ########## @@ -65,4 +94,33 @@ public interface RateLimiter { * @param filterConfig The FilterConfig used to configure the associated filter */ void setFilterConfig(FilterConfig filterConfig); -} + + /** + * Minimum set of rate limit header fields. + * + * @see <a HREF= + * "https://www.ietf.org/archive/id/draft-ietf-httpapi-ratelimit-headers-08.html#name-service-limit-item">ietf + * Ratelimit Headers Draft</a> + */ + public static class RateLimitItem { + private String policyName; + private int remainingQuota; + + /** + * @param name of policy associated + * @param remaining quota units for the identified policy + * @param timeLeftUntilReset number of seconds until the quota associated with the quota policy resets + */ + public RateLimitItem(String policyName, int remainingQuota) { + this.policyName = policyName; + this.remainingQuota = remainingQuota; + } + + @Override + public String toString() { + StringBuffer buf = new StringBuffer(); + buf.append('"').append(this.policyName).append('"').append(";r=").append(this.remainingQuota); Review Comment: `String.format()` will make the code much more readable here -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org