Skip to content

78 add rate limiting to the api #204

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 373 commits into
base: main
Choose a base branch
from
Open

78 add rate limiting to the api #204

wants to merge 373 commits into from

Conversation

bluedelaw
Copy link

Added rate limiting to the API. On line 99 of sec/Api/Extensions/ServiceExtensions.

If users request from the api more than 5 times within a 10-second window, the user will be rate limited for 10 seconds.

medhatelmasry and others added 30 commits June 24, 2024 17:16
@medhatelmasry
fixed authentication scheme issues
e9beacd
@medhatelmasry
GoodBooks light color on main page
a9d88a1
@medhatelmasry
No Warn in sahil branch
a5a6c58
@medhatelmasry
minutes of meeting with accountant on June 24, 2024
e27e967
@medhatelmasry
fixed issue 25
7aaa7db
@medhatelmasry
closed issue 19
422e197
@medhatelmasry
added blazor to mvc project
b4d2bbe
@medhatelmasry
added counter blazor component
491085f
@ydongchoi
[fix] Modified GoodBooks to Good Deed Books.
d675496
@medhatelmasry
Merge pull request #29 from medhatelmasry/16-change-app-name-to-good-…
cc1b382 
…deed-books

[Fix issue] Change app name to Good Deed Books
@medhatelmasry
fixed save customer and home page logo
5a3a476
@medhatelmasry
disabled react github actions workflow file
e8be1a3
@ydongchoi
[chore] Spaces are left between the code for ease of understanding.
a6925d8
@ydongchoi
[fix] changed script src for bootstraps to solve problem about taxs p…
7fb0388 
…age appearing nothing.
@medhatelmasry
Merge pull request #30 from medhatelmasry/11-financials-taxes
ae63091 
[Fix issue] Financials >> Taxes : There are clickable columns on this page but appear to do nothing.
@ydongchoi
[fix] Modified Script src of knockout.
de69e06
@ydongchoi
Merge branch '11-financials-taxes' of https://github.com/medhatelmasr…
8481cbb 
…y/GoodBooks into 11-financials-taxes
@medhatelmasry
Merge pull request #31 from medhatelmasry/11-financials-taxes
7512823 
[Fix issue] Financials >> Taxes : There are clickable columns on this page but appear to do nothing.
@ydongchoi
[fix] Modified the anchor tag for the base tag for the purpose of int…
4835898 
…egrating mvc app and blazor.
@ydongchoi
Merge branch '11-financials-taxes' of https://github.com/medhatelmasr…
9870280 
…y/GoodBooks into 11-financials-taxes
@medhatelmasry
Merge pull request #32 from medhatelmasry/11-financials-taxes
7bc0cb3 
[Fix issue] Financials >> Taxes : Redirection to base_route/#settings
@ydongchoi
[fix] Changed bootstrap version to use Collapsible Element properly.
143699f
@ydongchoi
[fix] Modified data-toggle Attribute Name to use Bootstrap Version 5.
4f7c9a1
@ydongchoi
[feature] Created AccountViewModel to use in Chart of Accounts page.
ae1bc45
@ydongchoi
[fix] Changed model type from string to AccountViewModel in Financial…
013074c 
…sController.
@ydongchoi
[feature] Added Accounts view that has table & accordion feature.
1068dce
@ydongchoi
[fix] Added ChildAccounts of grandChildAccount in Accounts view.
0ca8f0b
@medhatelmasry
Merge pull request #33 from medhatelmasry/17-exand-collapse-chart-of-…
50d0e5d 
…accounts

[Feature issue] Exand & collapse chart of accounts
@ydongchoi
[feature] Added Authentication Service to protect api from unauthoriz…
d2c0354 
…ed user in Progarm.
@ydongchoi
[fix] Added await keyword in Signin Action Method.
3d316cb
mika-em and others added 30 commits December 4, 2024 17:50
@mika-em
implement add/edit/delete buttons and modals in chart of accounts UI
9d953eb
@mika-em
replace id with accountCode in account service
70ce1b0
@mika-em
Merge branch 'allow-accounts-to-be-editable' of https://github.com/me…
976bf9d 
…dhatelmasry/GoodBooks into allow-accounts-to-be-editable
@medhatelmasry
included blazor yml
41ec533
@mika-em
add documentation and polish CRUD operations on frontend and backend
cf3d231
@medhatelmasry
exclude blazor yml
006b351
@mika-em
polish chart razor page and register account service in api program.cs
fa1a19d
@medhatelmasry
deleted app.db
b40900c
@medhatelmasry
deleted good-deed-books.db
93224c6
@medhatelmasry
moved endpoiny_sahil to main
80e3198
@medhatelmasry
dotnet 9
4081b76
@medhatelmasry
Merge pull request #74 from medhatelmasry/dotnet9
3561e28 
dotnet 9
@medhatelmasry
aspire workload in yml files
35727f9
@medhatelmasry
Merge pull request #75 from medhatelmasry/dotnet9
0d61e61 
aspire workload in yml files
@medhatelmasry
changed 8.0 to 9.0 on dotnet publish line in api yml
5616174
@medhatelmasry
Merge pull request #76 from medhatelmasry/dotnet9
84a39ae 
changed 8.0 to 9.0 on dotnet publish line in api yml
@medhatelmasry
fixed aspire.net
58d2e4d
@medhatelmasry
Got Aspire working with SQL Server in an automated fashion with a wor…
cb1344f 
…ker process
@medhatelmasry
Merge pull request #80 from medhatelmasry/automate-SQL-Server-in-Aspi…
f381945 
…re-Solution

Got Aspire working with SQL Server in an automated fashion with a wor…
@medhatelmasry
Merge pull request #79 from medhatelmasry/51-fix-allocations
9b72ab3 
51 fix allocations
@medhatelmasry
enabled exiry of API token as per Juan Chung and Ran Park research
488415e
@medhatelmasry
css for transaction row select and hover
7a19af5
@medhatelmasry
3_InitialData-0001-Audit.sql
f496829
@medhatelmasry
Merge pull request #81 from medhatelmasry/allow-accounts-to-be-editable
2fd2704 
Allow accounts to be editable
@medhatelmasry
updated all packages to latest and upgraded aspite to .net 9.0
b5aa8a1
@medhatelmasry
Merge pull request #85 from medhatelmasry/winter2025
8b03a53 
updated all packages to latest and upgraded aspire to .net 9.0
@TommyJu
Fix NullReference error when saving quotation
0ac64d8
@TommyJu
Debug invalid model state when saving a quotation
f939c89
@medhatelmasry
Merge pull request #91 from medhatelmasry/86-error-saving-quotation
b24aece 
86 error saving quotation
@bluedelaw
Added rate limiting to the API. After requesting 5 times, there will …
0d58328 
…be a 10 second timeout.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.