Also remove rate limit from /api/report endpoint.

Author: isoos

app/lib/frontend/handlers/report.dart

@@ -11,7 +11,6 @@ import 'package:pub_dev/admin/backend.dart';
 import 'package:pub_dev/shared/configuration.dart';
 import 'package:shelf/shelf.dart' as shelf;
 
-import '../../../service/rate_limit/rate_limit.dart';
 import '../../account/backend.dart';
 import '../../admin/models.dart';
 import '../../frontend/handlers/cache_control.dart';
@@ -25,11 +24,6 @@ import '../../shared/handlers.dart';
 import '../request_context.dart';
 import '../templates/report.dart';
 
-/// The number of requests allowed over [_reportRateLimitWindow]
-const _reportRateLimit = 5;
-const _reportRateLimitWindow = Duration(minutes: 10);
-const _reportRateLimitWindowAsText = 'last 10 minutes';
-
 /// Handles GET /report
 Future<shelf.Response> reportPageHandler(shelf.Request request) async {
   final feedback = request.requestedUri.queryParameters['feedback'];
@@ -175,17 +169,6 @@ Future<Message> processReportPageHandler(
   shelf.Request request,
   ReportForm form,
 ) async {
-  final sourceIp = request.sourceIp;
-  if (sourceIp != null) {
-    await verifyRequestCounts(
-      sourceIp: sourceIp,
-      operation: 'report',
-      limit: _reportRateLimit,
-      window: _reportRateLimitWindow,
-      windowAsText: _reportRateLimitWindowAsText,
-    );
-  }
-
   final now = clock.now().toUtc();
   final caseId = ModerationCase.generateCaseId(now: now);
 

app/lib/service/rate_limit/models.dart

@@ -6,7 +6,6 @@ import 'package:clock/clock.dart';
 import 'package:collection/collection.dart';
 import 'package:logging/logging.dart';
 import 'package:pub_dev/audit/backend.dart';
-import 'package:pub_dev/service/rate_limit/models.dart';
 
 import '../../account/agent.dart';
 import '../../audit/models.dart';
@@ -184,24 +183,3 @@ bool _containsUserId(List<String>? users, String userId) {
   }
   return users.contains(userId);
 }
-
-Future<void> verifyRequestCounts({
-  required String sourceIp,
-  required String operation,
-  required int limit,
-  required Duration window,
-  required String windowAsText,
-}) async {
-  final counterCacheEntry = cache.rateLimitRequestCounter(sourceIp, operation);
-  final cachedCounter = await counterCacheEntry.get();
-  if (cachedCounter == null) {
-    await counterCacheEntry.set(RateLimitRequestCounter.init(1));
-  } else {
-    final nextCounter = cachedCounter.incrementOrThrow(
-      limit: limit,
-      window: window,
-      windowAsText: windowAsText,
-    );
-    await counterCacheEntry.set(nextCounter);
-  }
-}

app/lib/service/rate_limit/models.g.dart

@@ -17,7 +17,6 @@ import 'package:neat_cache/neat_cache.dart';
 import 'package:pub_dev/service/download_counts/download_counts.dart';
 
 import '../../../service/async_queue/async_queue.dart';
-import '../../../service/rate_limit/models.dart';
 import '../../../service/security_advisories/models.dart';
 import '../../dartdoc/models.dart';
 import '../../shared/env_config.dart';
@@ -213,24 +212,6 @@ class CachePatterns {
         )[url];
   }
 
-  Entry<RateLimitRequestCounter> rateLimitRequestCounter(
-    String sourceIp,
-    String operation,
-  ) {
-    return _cache
-        .withPrefix('rate-limit-request-counter/')
-        .withTTL(const Duration(minutes: 3))
-        .withCodec(utf8)
-        .withCodec(json)
-        .withCodec(
-          wrapAsCodec(
-            encode: (RateLimitRequestCounter r) => r.toJson(),
-            decode: (d) =>
-                RateLimitRequestCounter.fromJson(d as Map<String, dynamic>),
-          ),
-        )['$sourceIp/$operation'];
-  }
-
   Entry<ScoreCardData> scoreCardData(String package, String version) => _cache
       .withPrefix('scorecard-data/')
       .withTTL(Duration(hours: 2))