Merge pull request #160 from cristipufu/fix/token_buckets_multi_permits #305
cristipufuAnnotations
11 warnings
|
Build and analyze
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-java@v3, actions/checkout@v3, actions/cache@v3, actions/setup-dotnet@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Build and analyze:
src/RedisRateLimiting/Concurrency/RedisConcurrencyRateLimiter.cs#L158
Await DisposeAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)
|
|
Build and analyze:
src/RedisRateLimiting/Concurrency/RedisConcurrencyRateLimiter.cs#L184
Await DisposeAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)
|
|
Build and analyze:
src/RedisRateLimiting/Concurrency/RedisConcurrencyRateLimiter.cs#L158
Await DisposeAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)
|
|
Build and analyze:
src/RedisRateLimiting/Concurrency/RedisConcurrencyRateLimiter.cs#L184
Await DisposeAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)
|
|
Build and analyze:
test/RedisRateLimiting.Sample.AspNetCore/Migrations/20221113165632_DatabaseStructure.cs#L18
Define a constant instead of using this literal 'RateLimits' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Build and analyze:
test/RedisRateLimiting.Sample.AspNetCore/Migrations/20221113165632_DatabaseStructure.cs#L21
Define a constant instead of using this literal 'INTEGER' 5 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Build and analyze:
test/RedisRateLimiting.Sample.AspNetCore/Migrations/20221113165632_DatabaseStructure.cs#L33
Define a constant instead of using this literal 'Clients' 4 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Build and analyze:
test/RedisRateLimiting.Sample.AspNetCore/Samples/ClientIdRateLimiterPolicy.cs#L43
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
Build and analyze:
test/RedisRateLimiting.Sample.AspNetCore/Program.cs#L12
Await ConnectAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)
|
|
Build and analyze:
test/RedisRateLimiting.Sample.AspNetCore/Program.cs#L80
Await EnsureCreatedAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)
|