Skip to content

fix: flow and forms request throttling #1133

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

Merged
merged 1 commit into from
Jul 28, 2025
Merged

Conversation

kushalshit27
Copy link
Contributor

@kushalshit27 kushalshit27 commented Jul 24, 2025

🔧 Changes

Improve request handling in Flow and Forms handlers by implementing throttling and optimizing data retrieval methods. This change enhances performance and reduces the number of requests made to the server.

📚 References

  • ESD-49365

🔬 Testing

Test pass ✅

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@kushalshit27 kushalshit27 changed the title fix: Flow and Forms request throttling fix: flow and forms request throttling Jul 24, 2025
@kushalshit27 kushalshit27 marked this pull request as ready for review July 24, 2025 13:58
@kushalshit27 kushalshit27 requested a review from a team as a code owner July 24, 2025 13:58
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 88.23529% with 2 lines in your changes missing coverage. Please review.

Project coverage is 80.29%. Comparing base (ebf361c) to head (89ba398).

Files with missing lines Patch % Lines
src/tools/auth0/handlers/flows.ts 87.50% 0 Missing and 1 partial ⚠️
src/tools/auth0/handlers/forms.ts 87.50% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1133      +/-   ##
==========================================
- Coverage   80.30%   80.29%   -0.01%     
==========================================
  Files         131      131              
  Lines        4956     4964       +8     
  Branches      976      978       +2     
==========================================
+ Hits         3980     3986       +6     
  Misses        563      563              
- Partials      413      415       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kushalshit27 kushalshit27 merged commit c4e4ccd into master Jul 28, 2025
8 checks passed
@kushalshit27 kushalshit27 deleted the DXCDT-1124-ESD-49365 branch July 28, 2025 06:30
@kushalshit27 kushalshit27 mentioned this pull request Jul 30, 2025
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.

3 participants