Skip to content

Commit c333302

Browse files
warwickschroederjasontaylordev
authored andcommitted
Remove previously added rate limit for anon api
1 parent 44ef6da commit c333302

File tree

2 files changed

+0
-21
lines changed

2 files changed

+0
-21
lines changed

src/ServiceControl/Authentication/AuthenticationController.cs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,14 @@
22
{
33
using Microsoft.AspNetCore.Authorization;
44
using Microsoft.AspNetCore.Mvc;
5-
using Microsoft.AspNetCore.RateLimiting;
65
using ServiceBus.Management.Infrastructure.Settings;
7-
using ServiceControl.Infrastructure.WebApi;
86

97
[ApiController]
108
[Route("api/authentication")]
119
public class AuthenticationController(Settings settings) : ControllerBase
1210
{
1311
[HttpGet]
1412
[AllowAnonymous]
15-
[EnableRateLimiting(HostApplicationBuilderExtensions.AuthConfigRateLimitPolicy)]
1613
[Route("configuration")]
1714
public ActionResult<AuthConfig> Configuration()
1815
{

src/ServiceControl/Infrastructure/WebApi/HostApplicationBuilderExtensions.cs

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,9 @@
11
namespace ServiceControl.Infrastructure.WebApi
22
{
3-
using System;
43
using System.Linq;
54
using System.Reflection;
6-
using System.Threading.RateLimiting;
75
using CompositeViews.Messages;
86
using Microsoft.AspNetCore.Builder;
9-
using Microsoft.AspNetCore.RateLimiting;
107
using Microsoft.Extensions.DependencyInjection;
118
using Microsoft.Extensions.DependencyInjection.Extensions;
129
using Microsoft.Extensions.Hosting;
@@ -15,8 +12,6 @@
1512

1613
static class HostApplicationBuilderExtensions
1714
{
18-
public const string AuthConfigRateLimitPolicy = "AuthConfigRateLimit";
19-
2015
public static void AddServiceControlApi(this IHostApplicationBuilder builder, CorsSettings corsSettings)
2116
{
2217
// This registers concrete classes that implement IApi. Currently it is hard to find out to what
@@ -27,19 +22,6 @@ public static void AddServiceControlApi(this IHostApplicationBuilder builder, Co
2722

2823
builder.Services.AddCors(options => options.AddDefaultPolicy(Cors.GetDefaultPolicy(corsSettings)));
2924

30-
// Rate limiting for sensitive endpoints to prevent enumeration attacks
31-
builder.Services.AddRateLimiter(options =>
32-
{
33-
options.AddFixedWindowLimiter(AuthConfigRateLimitPolicy, limiterOptions =>
34-
{
35-
limiterOptions.PermitLimit = 10;
36-
limiterOptions.Window = TimeSpan.FromMinutes(1);
37-
limiterOptions.QueueProcessingOrder = QueueProcessingOrder.OldestFirst;
38-
limiterOptions.QueueLimit = 2;
39-
});
40-
options.RejectionStatusCode = 429;
41-
});
42-
4325
// We're not explicitly adding Gzip here because it's already in the default list of supported compressors
4426
builder.Services.AddResponseCompression();
4527
var controllers = builder.Services.AddControllers(options =>

0 commit comments

Comments
 (0)