Skip to content

Commit 195dfef

Browse files
committed
TD_5410 Changes after discussion, removing IConfiguration and Extra json parameters from IPRateLimitor
1 parent 22a9fa2 commit 195dfef

File tree

2 files changed

+1
-18
lines changed

2 files changed

+1
-18
lines changed

DigitalLearningSolutions.Web/Middleware/DLSIPRateLimitMiddleware.cs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ public DLSIPRateLimitMiddleware(
1616
RequestDelegate next,
1717
IProcessingStrategy processingStrategy,
1818
IOptions<IpRateLimitOptions> options,
19-
IConfiguration configuration,
2019
IIpPolicyStore policyStore,
2120
IRateLimitConfiguration config,
2221
ILogger<IpRateLimitMiddleware> logger)
@@ -27,7 +26,6 @@ public DLSIPRateLimitMiddleware(
2726
config,
2827
logger)
2928
{
30-
_configuration = configuration;
3129
}
3230

3331
public override Task ReturnQuotaExceededResponse(

DigitalLearningSolutions.Web/appSettings.UAT.json

Lines changed: 1 addition & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -25,22 +25,7 @@
2525
"LoginWithLearningHub": true
2626
},
2727
"IpRateLimiting": {
28-
"EnableEndpointRateLimiting": false,
29-
"StackBlockedRequests": false,
30-
"RealIpHeader": "X-Real-IP",
31-
"HttpStatusCode": 429,
32-
"GeneralRules": [
33-
{
34-
"Endpoint": "post:/ForgotPassword",
35-
"Period": "1m",
36-
"Limit": 5
37-
},
38-
{
39-
"Endpoint": "post:/Login",
40-
"Period": "1m",
41-
"Limit": 5
42-
}
43-
]
28+
"EnableEndpointRateLimiting": false
4429
},
4530
"LearningHubOpenAPIBaseUrl": "https://uks-learninghubnhsuk-openapi-test.azurewebsites.net",
4631
"FreshdeskAPIConfig": {

0 commit comments

Comments
 (0)