Skip to content
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.

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

chore: limit auth status inactivate requests #5211

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sanpj2292
Copy link
Contributor

@sanpj2292 sanpj2292 commented Oct 21, 2024

Description

As of present day, we are receiving a lot of requests for inactivating auth status. The auth status inactivation happens only when the integration responds with permission denial when accessing a resource kind of error.
In order to acheive limiting, whenever we get an auth status inactivation request, we would use an in-memory cache to store the status of inactivation for the first such request & the next such request will be checking if that destination is present in cache & skip making an API call altogether.

Note:

  • The meaning of Inactivation is just updating the status to inactive

Linear Ticket

Resolves INT-2744

Security

  • The code changed/added as part of this pull request won't create any security issues with how the software is being used.

@sanpj2292 sanpj2292 self-assigned this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant